Closed
Bug 428097
Opened 16 years ago
Closed 16 years ago
Bookmarks Toolbar menu item in Bookmarks Menu doesn't have a menu icon
Categories
(Firefox :: Theme, defect, P2)
Tracking
()
VERIFIED
FIXED
Firefox 3
People
(Reporter: adelfino, Assigned: mak)
Details
(Keywords: regression)
Attachments
(2 files, 1 obsolete file)
82.59 KB,
image/png
|
Details | |
1.39 KB,
patch
|
dietrich
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008040907 Minefield/3.0pre Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008040907 Minefield/3.0pre Bookmarks Toolbar menu item in Bookmarks Menu doesn't have a menu icon. Reproducible: Always Steps to Reproduce: 1. Open the Bookmarks menu. 2. See the Bookmarks Toolbar menu item. Actual Results: Bookmarks Toolbar has no icon. Expected Results: Bookmarks Toolbar should have an icon.
Reporter | ||
Updated•16 years ago
|
Version: unspecified → Trunk
Reporter | ||
Comment 1•16 years ago
|
||
Reporter | ||
Comment 2•16 years ago
|
||
Nominating as blocking since all this work around Library should be as consistent as possible, since it's new stuff that the user will experiment with.
Flags: blocking-firefox3?
Comment 3•16 years ago
|
||
I see this on Win Vista as well, confirming using Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9pre) Gecko/2008040907 Minefield/3.0pre.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 4•16 years ago
|
||
Blocking because this is a regression - we used to show an icon here, what's going on? Marco, could this be related to your checkin from bug 421483?
Assignee | ||
Comment 5•16 years ago
|
||
doubt it, i simply created some bookmark and moved a separator...
Assignee | ||
Updated•16 years ago
|
Assignee: nobody → mak77
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Comment 7•16 years ago
|
||
This is WFM on Mac. Isn't auto the default value for -moz-image-region? Seems like you shouldn't need to specify it at all...
Assignee | ||
Comment 8•16 years ago
|
||
on win the style def is at the end of the file so probably region is defined in a previous styling (probably the generic container). will check later probably could be solved moving up the style in the css
Assignee | ||
Comment 9•16 years ago
|
||
so, after a check, the fix is only needed on winstripe since is the only theme that uses moz-region for bookmark-item, and being toolbar folder a bookmark-item[container] its region needs to be overloaded with auto.
Attachment #314871 -
Attachment is obsolete: true
Attachment #314979 -
Flags: review?(dietrich)
Attachment #314871 -
Flags: review?(dietrich)
Updated•16 years ago
|
Priority: -- → P2
Target Milestone: --- → Firefox 3
Comment 10•16 years ago
|
||
Comment on attachment 314979 [details] [diff] [review] patch r=me, thanks!
Attachment #314979 -
Flags: review?(dietrich) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #314979 -
Flags: approval1.9?
Comment 11•16 years ago
|
||
Comment on attachment 314979 [details] [diff] [review] patch a1.9=beltzner
Attachment #314979 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Comment 12•16 years ago
|
||
Checking in browser/themes/winstripe/browser/browser.css; /cvsroot/mozilla/browser/themes/winstripe/browser/browser.css,v <-- browser.css new revision: 1.197; previous revision: 1.196 done
Reporter | ||
Updated•16 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•