From d07bdd92a18eb894ce5300611c1ad9b6babf97ba Mon Sep 17 00:00:00 2001
From: Max Milton <max@happyhackers.xyz>
Date: Fri, 28 Jun 2024 10:01:45 +0900
Subject: [PATCH] feat: Remove `rich-black` theme

---
 src/css/themes/rich-black.xcss | 14 --------------
 src/settings.ts                |  1 -
 src/sw.ts                      | 11 +++++++++++
 test/unit/index.test.ts        |  2 +-
 4 files changed, 12 insertions(+), 16 deletions(-)
 delete mode 100644 src/css/themes/rich-black.xcss

diff --git a/src/css/themes/rich-black.xcss b/src/css/themes/rich-black.xcss
deleted file mode 100644
index 06a6b0710..000000000
--- a/src/css/themes/rich-black.xcss
+++ /dev/null
@@ -1,14 +0,0 @@
-/**
- * Rich Black Theme
- */
-
-:root {
-  --s: 0 0 0 1px #30221d; // shadow
-  --b: #0a0000; // background
-  --t: #795548; // text
-  --c1: #1d1616; // bookmark node hover background, hr, button border, search input border
-  --c2: #0f0b0b; // bookmarks bar, menu dropdown, "load more" button
-  --c3: #392d2d; // icons, empty folder text, text fallback (headings, etc.)
-
-  color-scheme: dark;
-}
diff --git a/src/settings.ts b/src/settings.ts
index 784192e9c..dfe45d0b3 100644
--- a/src/settings.ts
+++ b/src/settings.ts
@@ -114,7 +114,6 @@ const meta = compile(`
       <select @theme>
         <option value=light>Light</option>
         <option value=dark>Dark</option>
-        <option value=rich-black>Rich Black</option>
         <option value=hacker-blue>Hacker Blue</option>
         <option value=hacker-pink>Hacker Pink</option>
         <option value=hacker-terminal>Hacker Terminal</option>
diff --git a/src/sw.ts b/src/sw.ts
index bee62fa52..957912c16 100644
--- a/src/sw.ts
+++ b/src/sw.ts
@@ -9,6 +9,17 @@ chrome.runtime.onInstalled.addListener(async () => {
     (res) => res.json() as Promise<ThemesData>,
   );
   const settings: Promise<UserStorageData> = chrome.storage.local.get();
+
+  // TODO: Remove once most users have updated.
+  // Migrate users on old rich-dark theme (removed in v0.23.0)
+  if ((await settings).tn === 'rich-dark') {
+    void chrome.storage.local.set({
+      t: (await themes).dark,
+      tn: 'dark',
+    });
+    return;
+  }
+
   void chrome.storage.local.set({
     t: (await themes)[(await settings).tn ?? 'dark'],
   });
diff --git a/test/unit/index.test.ts b/test/unit/index.test.ts
index a01598bf8..75d623a43 100644
--- a/test/unit/index.test.ts
+++ b/test/unit/index.test.ts
@@ -19,7 +19,7 @@ describe('dist files', () => {
     ['settings.css', 'text/css;charset=utf-8', 1000, 1500],
     ['settings.html', 'text/html;charset=utf-8', 150, 200],
     ['settings.js', 'text/javascript;charset=utf-8', 4000, 6000],
-    ['sw.js', 'text/javascript;charset=utf-8', 150, 250],
+    ['sw.js', 'text/javascript;charset=utf-8', 150, 300],
     ['themes.json', 'application/json;charset=utf-8'],
   ];