Closed Bug 545842 Opened 15 years ago Closed 15 years ago

Style menu buttons and impose a standard size on third-party toolbarbutton icons

Categories

(Firefox :: Theme, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 3.7a2

People

(Reporter: dao, Assigned: dao)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

Attachments

(4 files)

menu button example: adblock plus oversized-icon example: https://addons.mozilla.org/en-US/firefox/addon/60267
Summary: Need to style menu buttons and impose a standard size on third-party toolbarbutton icons → Style menu buttons and impose a standard size on third-party toolbarbutton icons
Attached patch patch (deleted) — Splinter Review
Attachment #426654 - Flags: review?(rflint)
Comment on attachment 426654 [details] [diff] [review] patch >+.toolbarbutton-menubutton-button, >+.toolbarbutton-menubutton-dropmarker, > .toolbarbutton-1 { [...] > border: 1px solid; > border-color: rgba(0,0,0,.1) rgba(0,0,0,.15) rgba(0,0,0,.25); Note that the dropmarker suffers from bug 482692. >+:not([open="true"]):not(:active):hover > .toolbarbutton-menubutton-dropmarker:not([disabled="true"]), I actually wanted .toolbarbutton-menubutton-dropmarker:hover, but this didn't work for some reason.
Depends on: 482692
Attachment #426654 - Flags: review?(rflint) → review+
Attached image Screenshot (deleted) —
Does not this patch support toolbarbutton[type="menu"] styleing? example: User Agent Switcher https://addons.mozilla.org/en-US/firefox/addon/59
(In reply to comment #3) > Created an attachment (id=426725) [details] > Screenshot > > Does not this patch support toolbarbutton[type="menu"] styleing? Currently only with the toolbarbutton-1 class.
Attached image Screenshot (deleted) —
Dao, my menu-buton looks different from the ABP menu-button and I'm wondering if this patch is going to fix it? The ABP button doesn't have any type of border on non-hover whilst mine does and even has the animation. My button just seems to me missing the black border where it is now white. Here is my code: https://addons.mozilla.org/en-US/firefox/files/browse/75741 Button code is under chrome -> content -> firefoxOverlay.xul Style code is under chrome -> skin -> overlay.css
Blocks: 546098
(In reply to comment #5) > Created an attachment (id=426845) [details] > Screenshot > > Dao, my menu-buton looks different from the ABP menu-button and I'm wondering > if this patch is going to fix it? https://build.mozilla.org/tryserver-builds/dgottwald@mozilla.com-try-a5f8a24b7ddd/try-a5f8a24b7ddd-win32.zip contains the patch.
Attached image Screenshot (undersized button) (deleted) —
Will this patch also fixed the issue of undresized buttons? The attached screenshot is for small icons on Windows XP with classic theme. Note how disproportionately small the Video DownloadHelper button is.
Also, notice, as show in the image labeled "Hover over ABP button" in the screenshot of Attachment 426845 [details], that there is an extra few pixels to the right over every button that has a drop-down. I notice the exact same issue with all such buttons on my toolbar.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a2
I have a black border to the right of the menu button portion a menu-button. Are all icons are now going to forced to be 16px? I just want to know so I can delete the code is so, plus a dev doc should be created to mention it. And for some reason with small icons code with icons set to be small, my extensions icon is fuzzy and looks stretched even though the icon is 16px. Code: toolbar[iconsize="small"] #undoclosedtabsbutton-toolbar-button {list-style-image: url ("chrome://undoclosedtabsbutton/skin/toolbar-button-small-passive.png");}
Icons are currently forced to be 18px, but only with the default theme on Windows...
Verified Fixed on Trunk using ABP, Video Download Helper, and Minimap. Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.3a2pre) Gecko/20100218 Minefield/3.7a2pre (.NET CLR 3.5.30729) ID:20100218045818
Status: RESOLVED → VERIFIED
Depends on: 547418
No longer depends on: 482692
Depends on: 547419
Depends on: 547420
Depends on: 547422
No longer depends on: 547419
Depends on: 547128
Blocks: 462512
Blocks: 578181
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: