johngraciliano / simplewhite

A minimal theme for Mozilla Firefox.
https://addons.mozilla.org/addon/simplewhitex/
Mozilla Public License 2.0
44 stars 4 forks source link

Wrong fit-size for the small bookmark icon. #8

Open johngraciliano opened 8 years ago

johngraciliano commented 8 years ago

At some moment, perhaps since my first release, I introduced a bug that incorrectly makes the area available to display the small bookmark icon 17x17 instead of 18x18. As a result the icon is scaled down and does not look as sharp (when 1dot=1pixel) as it did in the original theme. I however prefer the smaller icon because the original bookmark icon appears too large when compared to the the book (shown bookmarks) icon. I plan to revert the code to make the area for the bookmark icon be able to fit the 18x18 icon. However, I may change the icon to have a smaller (but sharp!) image later on. Feedback regarding this is requested

Please, see the attached image in a separate tab/window with a 100% zoom for a better detail. bookmark icon size2

johngraciliano commented 8 years ago

I removed the incorrect code from browser/browser-common.css. The problem is fixed. However, if no one objects, I will change the icon for a slightly smaller version when I change the toolbar icons resource to svg.