419760 - Update windows theme for 78 pixel wide keyhole

Status: RESOLVED FIXED

(Firefox :: Theme, defect, P1)

Description

[Alex Faaborg [:faaborg] (Firefox UX)]

I'm currently in the process of compositing together a new toolbar.png.  We would like to have the etch extend to the left side, making the entire keyhole 76 pixels wide. This breaks down horizontally as:

[back 36 pixels][forward 28 pixels][drop down 12 pixels]

the height will remain the same.

I'll post the new toolbar.png once it is done, but I wanted to get the sizes up since we are nearing freeze.

Comment 1

[Alex Faaborg [:faaborg] (Firefox UX)]

Attachment: example of the new 76 by 34 keyhole

Comment 2

[Alex Faaborg [:faaborg] (Firefox UX)]

sorry, made another change:

[back 38 pixels][forward 28 pixels][drop down 14 pixels]

Comment 3

[Alex Faaborg [:faaborg] (Firefox UX)]

Attachment: Example of a 78 by 34 keyhole

Comment 4

[Alex Faaborg [:faaborg] (Firefox UX)]

Attachment: Example toolbar.png (in progress)

Comment 5

[Alex Faaborg [:faaborg] (Firefox UX)]

Attachment: Example toolbar.png (still in progress, but should be nearly final)

Comment 6

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

mcdavis, chances are that you can start morphing the CSS required for this before we have the final image, right?

Comment 7

[Alex Faaborg [:faaborg] (Firefox UX)]

Attachment: Toolbar-aero.png

Here is the aero version

Comment 8

[mcdavis941 (sporadically reading bugmail)]

(In reply to comment #6)
> mcdavis, chances are that you can start morphing the CSS required for this
> before we have the final image, right?
> 
Right.  Alex asked if I could get this in this evening and got me started with dimensions earlier.  There's a work in progress patch coming shortly.

Comment 9

[mcdavis941 (sporadically reading bugmail)]

This will probably need some padding to keep it from actually touching the left edge of the window. See https://bugzilla.mozilla.org/show_bug.cgi?id=414842#c9

Comment 10

[mcdavis941 (sporadically reading bugmail)]

Attachment: browser.css changes v1

Not yet tested on RTL but expected to be good for both LTR and RTL versions.

Comment 11

[mcdavis941 (sporadically reading bugmail)]

Ready for review.

Comment 12

[mcdavis941 (sporadically reading bugmail)]

(In reply to comment #11)
> Ready for review.

Was unable to test for RTL -- I tried but a stock install of the 02-26 Hebrew nightly couldn't get past crash-on-launch, even without any changes by me -- so maybe someone else could look at this from an RTL perspective?  If no one does, I'll have another go at this tomorrow.

I'm asking for review anyway because I think Alex wants this in.

Comment 13

[timeless]

is there a bug tracking the crash? such things need to be investigated...

Comment 14

[mcdavis941 (sporadically reading bugmail)]

(In reply to comment #13)
> is there a bug tracking the crash? such things need to be investigated...
> 
Filed Bug 419966

There's also Bug 415531, although it doesn't seem necessarily related.

Comment 15

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

Comment on attachment 305959 [details] [diff] [review]
browser.css changes v1

a1.9b4=beltzner

This needs to land with the ZIP file (coming soon!) with new icon files in bug 414389

Comment 16

[Reed Loden [:reed] (use needinfo?)]

Checking in browser/themes/winstripe/browser/browser.css;
/cvsroot/mozilla/browser/themes/winstripe/browser/browser.css,v  <--  browser.css
new revision: 1.182; previous revision: 1.181
done

Comment 17

[Peter S.]

Something I recently found: (http://forums.mozillazine.org/viewtopic.php?p=3277517#3277517)

The back button looks like it's lower than it possibly wants to be. I'll try to map it out:


File Edit
--------
(space)
(back button)
--bottom of toolbar--


The top of the back button does not scrape against the top of the toolbar, while the bottom of the back button does scrape against the bottom of the toolbar.

Intended? It looks like some re-positioning of the back/forward button may need to happen if things are to look perfectly aligned.  Look here for more information:

Edit
-------
(some space)
-----keyhole
(space a)
(forward button)
(space a)
-----keyhole
bottom of toolbar

I -think- "space a" above and below the forward button are both the same, but if you look closely below the Edit option, it looks like the heyhole is positioned too far down.

Comment 18

[Peter S.]

Possible I spoke to soon with the above note:

I was directed towards bug 415329 - large toolbar buttons are vertically misaligned.
https://bugzilla.mozilla.org/show_bug.cgi?id=415329