Closed
Bug 425625
Opened 16 years ago
Closed 16 years ago
"Open All In Tabs" doesn't fit with the New Tab icon
Categories
(Firefox :: Theme, defect)
Tracking
()
RESOLVED
FIXED
Firefox 3
People
(Reporter: ventnor.bugzilla, Assigned: ventnor.bugzilla)
References
Details
Attachments
(1 file)
|
1.05 KB,
patch
|
asaf
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
a) It doesn't exactly fit with the metaphor, but the much more important reason: b) The menu item disables on smart folders, yet there is no CSS rule or image to cater for this case. We should remove the use of that icon on this menu item.
Attachment #312232 -
Flags: review?(mano)
|
||
Comment 1•16 years ago
|
||
Comment on attachment 312232 [details] [diff] [review] Patch r=mano
Attachment #312232 -
Flags: review?(mano) → review+
|
||
Comment 2•16 years ago
|
||
Comment on attachment 312232 [details] [diff] [review] Patch Simple one-line CSS removal to remove menu icon that doesn't fit correctly.
Attachment #312232 -
Flags: approval1.9?
|
||
Comment 3•16 years ago
|
||
Comment on attachment 312232 [details] [diff] [review] Patch a=beltzner
Attachment #312232 -
Flags: approval1.9? → approval1.9+
|
|
||
Updated•16 years ago
|
Keywords: checkin-needed
|
|
||
Comment 4•16 years ago
|
||
Checking in browser/themes/gnomestripe/browser/places/places.css; /cvsroot/mozilla/browser/themes/gnomestripe/browser/places/places.css,v <-- places.css new revision: 1.30; previous revision: 1.29 done
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3
You need to log in
before you can comment on or make changes to this bug.
Description
•