Closed Bug 432757 Opened 14 years ago Closed 14 years ago

darken .ac-url-title for vista

Categories

(Firefox :: Theme, defect)

x86
Windows Vista
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: beltzner, Assigned: beltzner)

Details

Attachments

(3 files, 1 obsolete file)

Attached patch change to #0055bb (obsolete) — Splinter Review
.ac-url-title was switched from green (#006600) to a light blue (#0066cc) on Vista since green isn't a colour that's used all that often, and that light blue is what's used in the Vista UI for links.

Trouble is, in the context of the awesomebar, that's a lot of light blue, and especially on large monitors it ends up feeling very overpowering.

We should darken the colour. I tried out a bunch of hues, and a simple darkening to 0055bb ended up looking pretty good.
Attachment #319912 - Flags: review?(gavin.sharp)
Whoops. This is the right patch.
Attachment #319912 - Attachment is obsolete: true
Attachment #319913 - Flags: review?(gavin.sharp)
Attachment #319912 - Flags: review?(gavin.sharp)
Comment on attachment 319913 [details] [diff] [review]
change to #0055bb

Are those the right screenshots? I can't tell the difference between them...
Attachment #319913 - Flags: review?(gavin.sharp) → review+
Comment on attachment 319913 [details] [diff] [review]
change to #0055bb

eh, just land it.  Its definitely a darker color... :)
Attachment #319913 - Flags: approval1.9+
Assignee: nobody → beltzner
mozilla/browser/themes/winstripe/browser/browser-aero.css 	1.17 
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3
(In reply to comment #4)
> (From update of attachment 319913 [details] [diff] [review])
> Are those the right screenshots? I can't tell the difference between them...

Yeah, it's really subtle, but makes a difference with really bright screens. Thanks, gents!
Target Milestone: Firefox 3 → ---
You need to log in before you can comment on or make changes to this bug.