Use themed button on 32px as well

denschub · Apr 29, 2016 · 54b784e · 54b784e
1 parent bebc291
commit 54b784e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -1,5 +1,9 @@
 # 0.5.0
 
+## Bug fixes
+
+* Use the inverted toolbar icon if needed on high resolutions.
+
 ## Refactorings
 
 * Use the default favicon instead of no favicon at all. (PR [35](https://github.com/denschub/firefox-tabgroups/pull/35))

diff --git a/src/index.js b/src/index.js
@@ -57,14 +57,16 @@ TabGroups.prototype = {
 
   createPanelButton: function() {
     let iconBase = "chrome://simplified-tabgroups/content/icons/togglebutton/";
+    let toolBarButton = Utils.themeSwitch({
+      dark: iconBase + "icon-inverted-32.png",
+      light: iconBase + "icon-32.png"
+    });
+
     this._panelButton = ToggleButton({
       id: "tabgroups-show",
       icon: {
-        "16": Utils.themeSwitch({
-          dark: iconBase + "icon-inverted-32.png",
-          light: iconBase + "icon-32.png"
-        }),
-        "32": iconBase + "icon-32.png",
+        "16": toolBarButton,
+        "32": toolBarButton,
         "64": iconBase + "icon-64.png"
       },
       label: _("panelButton_label")