firefox-csshacks
firefox-csshacks copied to clipboard
create toolbar_icons_hover_zoom.css
/* This stylesheet makes main toolbar buttons nicely zoom on hover. */
This is pretty neat effect, good job!
I can merge this, but I'm wondering why you are limiting the effect to only buttons in the nav-bar? I think it should apply to buttons regardless of which toolbar they are in.
Also, why the drop-shadows? I think if the user wants drop shadows for icons, they should use a style that does just that, and let this style be just what the name implies.
Which brings me to another issue, the name of the style. Or rather, there is already button_effect_scale_onhover.css, and that name is more fitting to this one I think. Unfortunately we can't merge these two sytles because they do different things. The old one scales just the icon, but this yours does scale the whole button.
Lastly, can I add the normal file header to this style? It would say:
/* Source file https://github.com/MrOtherGuy/firefox-csshacks/tree/master/chrome/toolbar_icons_hover_zoom.css made available under Mozilla Public License v. 2.0
See the above repository for updates as well as full license text. */
I want to include those mainly to inform whoever uses the style about where they can find the source but also to inform them about MPLv2 license.
So, if you don't have anything against it I would like to
- Make this apply to buttons in all toolbar, not just nav-bar.
- remove filter effects (drop-shadow)
- Change the file-name to something else.
- Add the normal header that is found in other styles.
I can do all of the above changes after merging if you prefer that. But do tell if you have strong opinions about why some of those should not be done.