fix: do not actually use OptionsManagement.isDarkMode as it does not …

…exist
gyng · Feb 13, 2023 · 8960e02 · 8960e02
1 parent bc0b7b1
commit 8960e02
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+# 3.7.2
+
+* Fix light mode icon again
+
 # 3.7.1
 
 * Fix invalid default menu configuration

diff --git a/manifest.json b/manifest.json
@@ -2,7 +2,7 @@
  "manifest_version": 2,
  "name": "__MSG_extensionName__",
  "description": "__MSG_extensionDescription__",
- "version": "3.7.1",
+ "version": "3.7.2",
  "default_locale": "en",
 
  "applications": {

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "save-in",
- "version": "3.7.1",
+ "version": "3.7.2",
  "license": "MIT",
  "scripts": {
  "build": "env -u WEB_EXT_API_KEY -u WEB_EXT_API_SECRET web-ext build --overwrite-dest -i test docs yarn.lock yarn-error.log",

diff --git a/src/menu.js b/src/menu.js
@@ -156,9 +156,7 @@ const Menus = {
  browser.contextMenus.create(
  Object.assign({}, lastUsedMenuOptions, {
  icons: {
- 16: OptionsManagement.isDarkMode()
- ? "icons/ic_update_light_24px.svg"
- : "icons/ic_update_black_24px.svg",
+ 16: "icons/ic_update_black_24px.svg",
  },
  })
  );