Closed
Bug 430384
Opened 17 years ago
Closed 17 years ago
tweak browser.css to use new dimensions of keyhole on windows xp & vista
Categories
(Firefox :: Theme, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3
People
(Reporter: beltzner, Assigned: dao)
Details
Attachments
(7 files)
Alex to provide new dimensions and new Toolbar.png and Toolbar-areo.png files.
Flags: blocking-firefox3+
Reporter | ||
Comment 1•17 years ago
|
||
(toolbar-aero.png, even)
Comment 2•17 years ago
|
||
Dao, hopefully by the time you read this, Alex will have this stuff posted. Please request review from me, since I'm on EDT, and we're trying to wrap up as soon as possible, this is one of the last blockers.
Assignee: nobody → dao
Reporter | ||
Comment 3•17 years ago
|
||
Sadly, we got a broken delivery from the IconFactory. However, Alex tells me that he can assemble a Toolbar.png which we can check in and base CSS changes on, and then we can replace that with another one (he'll file a follow-up bug and nominate for blocking) when we get the fixes.
Alex, can you post those new Toolbar.png and Toolbar-aero.png files ASAP so we can get the CSS part done?
Comment 4•17 years ago
|
||
Comment 5•17 years ago
|
||
Comment 6•17 years ago
|
||
Comment 7•17 years ago
|
||
Comment 8•17 years ago
|
||
These are unfortunately not the final files (drop marker is not disabled in the disabled states), however go ahead and land them with the css changes and we will refresh them with the next icon drop. The image regions will stay the same.
Assignee | ||
Comment 9•17 years ago
|
||
Attachment #317227 -
Flags: review?(mconnor)
Comment 10•17 years ago
|
||
> Toolbar.png
> Toolbar-small.png
Bug 429689 comment #5
I do not understand why only cut, copy and paste icons are different behavior.
Are there any guidelines?
Comment 11•17 years ago
|
||
I'm sorry.
My wanting to say is this.
Bug 429689 comment #10
Reporter | ||
Comment 12•17 years ago
|
||
AKIHIRO, please see comment 8 where Alex says these aren't the final images. I believe your comments will be addressed when we get all of those images.
Comment 13•17 years ago
|
||
Comment on attachment 317227 [details] [diff] [review]
patch
r+a=me, thanks!
Attachment #317227 -
Flags: review?(mconnor)
Attachment #317227 -
Flags: review+
Attachment #317227 -
Flags: approval1.9+
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Reporter | ||
Updated•17 years ago
|
Whiteboard: [has patch][has reviews][has approval]
Comment 14•17 years ago
|
||
mozilla/browser/themes/winstripe/browser/Toolbar-aero.png 1.6
mozilla/browser/themes/winstripe/browser/Toolbar-small-aero.png 1.4
mozilla/browser/themes/winstripe/browser/Toolbar-small.png 1.11
mozilla/browser/themes/winstripe/browser/Toolbar.png 1.12
mozilla/browser/themes/winstripe/browser/browser.css 1.206
Status: NEW → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [has patch][has reviews][has approval]
Target Milestone: --- → Firefox 3
Comment 15•17 years ago
|
||
(In reply to comment #12)
> AKIHIRO, please see comment 8 where Alex says these aren't the final images. I
> believe your comments will be addressed when we get all of those images.
OK, and more,
{Build ID: 2008042400}
Toolbar.png (New tab, New Window) and Toolbar-small.png (Print, New tab, New Window).
Mistake of arrangement
Comment 16•17 years ago
|
||
APNG Screenshot
Comment 17•17 years ago
|
||
Not sure if this is the right place to bring this up, but I personally have mixed feelings about the new toolbar icons. To avoid taking this off-topic, I'll just post a link:
http://forums.mozillazine.org/viewtopic.php?p=3350238#3350238
Comment 18•17 years ago
|
||
Uhm, last icons look more bad IMHO, they are more dark than the previous.
And the keyhole is more vertical unaligned now.
Comment 19•17 years ago
|
||
The aero small disabled forwards button got shifted 1px with this checkin:
before: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/634491129642/browser/themes/winstripe/browser/Toolbar-small-aero.png
after: http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/raw-file/8f8ce43692b4/browser/themes/winstripe/browser/Toolbar-small-aero.png
Comment 20•17 years ago
|
||
I cannot see any of these mentioned aligning issues. The keyhole looks good to me and even the disabled forward button is positioned correctly now with Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9pre) Gecko/2008050206 Minefield/3.0pre
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•