574479 - Adjust Bookmarks Button styling and icons to match mockups

Status: NEW

(Firefox :: Theme, defect)

Description

[Peter Henkel [:Terepin]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a6pre) Gecko/20100623 Minefield/3.7a6pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a6pre) Gecko/20100623 Minefield/3.7a6pre

1. (Bookmarks Toolbar on) Change icon
2. (Bookmarks Toolbar off) Make button wider and drop arrow bigger
?. Change menu items

Reproducible: Always

Comment 1

[Marco Bonardo [:mak]]

ok we can probably use this for icon too

Comment 2

[Peter Henkel [:Terepin]]

3. (Bookmarks Toolbar on) Button doesn't stay in pressed state

Comment 3

[Peter Henkel [:Terepin]]

Upps, I mean "Bookmarks Toolbar off"

Comment 4

[Marco Bonardo [:mak]]

(In reply to comment #2)
> 3. (Bookmarks Toolbar on) Button doesn't stay in pressed state

that's bug 575162

Comment 5

[imradyurrad]

Blocking 578025?

Comment 6

[imradyurrad]

Or beta7? Inactivity sucks.

Comment 7

[Mike Beltzner [:beltzner, not reading bugmail]]

Taz and Peter: please create mockups/graphics like Stephen has done over at https://wiki.mozilla.org/Firefox/Projects/New_Theme/Timeline which indicate to pixel precision what changes you think should be made.

Stephen: please renominate if you think this should block. I can't really tell from this summary or comments if these are issues you feel are important, or what needs to be done, so can't block on this at the moment.

Comment 8

[Peter Henkel [:Terepin]]

Attachment: Bookmarks Bar on

Comment 9

[Peter Henkel [:Terepin]]

Attachment: Bookmarks Bar off

Note: Proper text will be changed in bug 575580.

Comment 10

[Peter Henkel [:Terepin]]

Stephen, I still consider this at least a softblocker. Can you create patch, please?