This bug will track landing what will hopefully be the final drop of windows icons before RC1.

These icons should be ready this evening.

Or perhaps this morning! :)

Yeah, emphasize on "should be" we have all of the files, I am working on preparing the drop now.

This is blocking since it fixes a blocker in that the back/forward dropmarker is always showing as enabled. Alex: can we get those images checked in immediately and then we can remove this from the blocker list? Unless there are other changes coming in this drop that you believe should block us. I don't really know if I have a full understanding of what this is changing.

Landing these 4 files will clear the blocking flag, but we should leave the bug open for the full drop.

Comment on attachment 318317 [details] New Toolbar files r+uir+a=beltzner

Comment on attachment 318317 [details] New Toolbar files Checking in browser/themes/winstripe/browser/Toolbar-aero.png; /cvsroot/mozilla/browser/themes/winstripe/browser/Toolbar-aero.png,v <-- Toolbar-aero.png new revision: 1.8; previous revision: 1.7 done Checking in browser/themes/winstripe/browser/Toolbar-small-aero.png; /cvsroot/mozilla/browser/themes/winstripe/browser/Toolbar-small-aero.png,v <-- Toolbar-small-aero.png new revision: 1.5; previous revision: 1.4 done Checking in browser/themes/winstripe/browser/Toolbar-small.png; /cvsroot/mozilla/browser/themes/winstripe/browser/Toolbar-small.png,v <-- Toolbar-small.png new revision: 1.13; previous revision: 1.12 done Checking in browser/themes/winstripe/browser/Toolbar.png; /cvsroot/mozilla/browser/themes/winstripe/browser/Toolbar.png,v <-- Toolbar.png new revision: 1.14; previous revision: 1.13 done

(In reply to comment #5) > Landing these 4 files will clear the blocking flag, but we should leave the bug > open for the full drop. Filed 431285 for the remainder.

rtl xp small icons were shifted 1px left xp small 4-25: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/87893472b320/browser/themes/winstripe/browser/Toolbar-small.png 4-28: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/878eb6749f48/browser/themes/winstripe/browser/Toolbar-small.png (open both in tabs and switch between the two) And if you wanted to just double check the rest.. aero small 4-25: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/87893472b320/browser/themes/winstripe/browser/Toolbar-small-aero.png 4-28: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/878eb6749f48/browser/themes/winstripe/browser/Toolbar-small-aero.png xp 4-25: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/87893472b320/browser/themes/winstripe/browser/Toolbar.png 4-28: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/878eb6749f48/browser/themes/winstripe/browser/Toolbar.png aero 4-25: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/87893472b320/browser/themes/winstripe/browser/Toolbar-aero.png 4-28: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/878eb6749f48/browser/themes/winstripe/browser/Toolbar-aero.png

A the disabled keyhole state was 1 pixel off in the last set of attachments, please land this update immediately.

Reopening to land updated file.

Comment on attachment 318330 [details] New Toolbar-small.png Checking in browser/themes/winstripe/browser/Toolbar-small.png; /cvsroot/mozilla/browser/themes/winstripe/browser/Toolbar-small.png,v <-- Toolbar-small.png new revision: 1.14; previous revision: 1.13 done

Hrmm... comparing xp and vista small icons: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/878eb6749f48/browser/themes/winstripe/browser/Toolbar-small-aero.png http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/878eb6749f48/browser/themes/winstripe/browser/Toolbar-small-aero.png aero's disabled forwards seems to shift left compared to xp

Sorry, should be http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/878eb6749f48/browser/themes/winstripe/browser/Toolbar-small.png http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/878eb6749f48/browser/themes/winstripe/browser/Toolbar-small-aero.png

Corrected an off by one pixel error with the disabled 16x16 forward button

Comment on attachment 318335 [details] Toolbar-small-aero.png Checking in browser/themes/winstripe/browser/Toolbar-small-aero.png; /cvsroot/mozilla/browser/themes/winstripe/browser/Toolbar-small-aero.png,v <-- Toolbar-small-aero.png new revision: 1.6; previous revision: 1.5 done