Closed
Bug 520819
Opened 16 years ago
Closed 16 years ago
header colors are too dark
Categories
(Thunderbird :: Message Reader UI, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0rc1
People
(Reporter: clarkbw, Assigned: andreasn)
References
Details
(Whiteboard: [no l10n impact])
Attachments
(2 files, 1 obsolete file)
351.24 KB,
image/png
|
Details | |
1.63 KB,
patch
|
andreasn
:
review+
andreasn
:
ui-review+
|
Details | Diff | Splinter Review |
much of the header theme colors for the mac were ported over from the windows and linux themes originally. We need to update these to be more inline with the mac theme, especially apps like Mail.app look a lot lighter in this respect.
The mac uses a much better a11y system than windows and linux so we don't need to stick to css system colors like we would for those. However we should figure out the mac theme colors and stay inline with that.
Flags: blocking-thunderbird3+
Updated•16 years ago
|
Whiteboard: [no l10n impact]
Assignee | ||
Comment 1•16 years ago
|
||
It seems like the actual background color is equally heavy as other mac apps, but I think we could make the buttons look nicer in order to look less 'heavy'.
Assignee | ||
Comment 2•16 years ago
|
||
This makes the header feel a lot lighter.
Also fixes the odd looking Other Actions-button.
Assignee | ||
Comment 3•16 years ago
|
||
Assignee | ||
Updated•16 years ago
|
Attachment #405481 -
Flags: ui-review?(clarkbw)
Assignee | ||
Updated•16 years ago
|
Attachment #405481 -
Flags: review?(dmose)
Updated•16 years ago
|
Attachment #405481 -
Flags: review?(dmose) → review+
Comment 4•16 years ago
|
||
Comment on attachment 405481 [details] [diff] [review]
patch for lighter header buttons
r=dmose
Reporter | ||
Comment 5•16 years ago
|
||
Comment on attachment 405481 [details] [diff] [review]
patch for lighter header buttons
this bit-rotted a little with the other mac header patch going in but it looks good.
Attachment #405481 -
Flags: ui-review?(clarkbw) → ui-review+
Assignee | ||
Comment 6•16 years ago
|
||
This should work!
Attachment #406436 -
Flags: ui-review+
Attachment #406436 -
Flags: review+
Assignee | ||
Updated•16 years ago
|
Keywords: polish → checkin-needed
Updated•16 years ago
|
Attachment #405481 -
Attachment is obsolete: true
Comment 7•16 years ago
|
||
Status: NEW → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•