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)

x86
Windows Vista
defect
Not set
normal

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+
(toolbar-aero.png, even)
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
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?
Attached image Toolbar-aero.png (deleted) —
Attached image Toolbar-small.png (deleted) —
Attached image Toolbar.png (deleted) —
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.
Attached patch patch (deleted) — Splinter Review
Attachment #317227 - Flags: review?(mconnor)
> 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?
I'm sorry. My wanting to say is this. Bug 429689 comment #10
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 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+
Keywords: checkin-needed
Whiteboard: [has patch][has reviews][has approval]
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
(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
Attached image Screenshot (deleted) —
APNG Screenshot
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
Uhm, last icons look more bad IMHO, they are more dark than the previous. And the keyhole is more vertical unaligned now.
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.

Attachment

General

Created:
Updated:
Size: