Issue 4409 - Fixed too large toolbar icon in older Firefox versions on high-DPI screens

chrome/skin/overlay.css

Index: chrome/skin/overlay.css
===================================================================
--- a/chrome/skin/overlay.css
+++ b/chrome/skin/overlay.css
@@ -39,16 +39,24 @@ toolbar[iconsize="small"] #abp-toolbarbu
 toolbar[iconsize="small"] #abp-toolbarbutton[abpstate="whitelisted"],
 #PersonalToolbar #abp-toolbarbutton[abpstate="whitelisted"],
 #header-view-toolbar > #abp-toolbarbutton[abpstate="whitelisted"],
 #abp-status[abpstate="whitelisted"]
 {
   -moz-image-region: rect(16px, 16px, 32px, 0px);
 }
 
+toolbar[iconsize="small"] #abp-toolbarbutton .toolbarbutton-icon,

[saroyanm, 2016/09/19 10:40:09]

According to the Ticket description, this affects only high DPI screens, so I
assume the change should apply only to high DPI screens ? Or this affects
regular screens as well ?

[Wladimir Palant, 2016/09/19 11:08:40]

On regular screens the image size happens to match the displayed size. However,
setting size explicitly won't hurt - and we avoid duplicating this code for
1.5dppx and 2.0dppx.

[saroyanm, 2016/09/19 11:57:22]

Acknowledged.

+#PersonalToolbar #abp-toolbarbutton .toolbarbutton-icon,
+#header-view-toolbar > #abp-toolbarbutton .toolbarbutton-icon
+{
+  width: 16px;
+  height: 16px;
+}
+
 @media screen and (min-resolution: 1.5dppx)
 {
   toolbar[iconsize="small"] #abp-toolbarbutton,
   #PersonalToolbar #abp-toolbarbutton,
   #header-view-toolbar > #abp-toolbarbutton,
   #abp-status
   {
     list-style-image: url("abp-status.png");