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

chrome/skin/overlay.css

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 26 matching lines @@
 #header-view-toolbar > #abp-toolbarbutton[abpstate="disabled"],
 #abp-status[abpstate="disabled"],
 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");
     -moz-image-region: rect(0px, 24px, 24px, 0px);
   }
@@ skipping 228 matching lines @@
 #abp-notification.abp-closing #abp-notification-content,
 #abp-notification:not(.abp-closing) #abp-notification-close-content
 {
   display: none;
 }
 
 #abp-notification-close-content toolbarbutton label
 {
   text-align: left;
 }