Closed
Bug 640090
Opened 13 years ago
Closed 13 years ago
Give account central system colors for non-Windows themes
Categories
(Thunderbird :: Theme, defect)
Thunderbird
Theme
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 5.0b1
People
(Reporter: squib, Assigned: squib)
References
Details
Attachments
(2 files, 1 obsolete file)
2.74 KB,
patch
|
bwinton
:
review+
bwinton
:
feedback+
|
Details | Diff | Splinter Review |
206.84 KB,
image/png
|
clarkbw
:
ui-review+
andreasn
:
feedback+
|
Details |
Bug 620467 gave account central system colors on Windows. We should probably do this for Mac and Linux too. Here's a patch for Linux. Not sure what the colors should be on Mac, since I can't really test them out...
Assignee | ||
Comment 1•13 years ago
|
||
bwinton/clarkbw/andreasn, do you have any ideas for the Mac version of this patch? I'm not sure what particular system colors I should be using.
Comment 2•13 years ago
|
||
(In reply to comment #1) > bwinton/clarkbw/andreasn, do you have any ideas for the Mac version of this > patch? I'm not sure what particular system colors I should be using. Tell you what, if you use the same colours you were using for Linux, I'll run it and take a screenshot for you… :) Thanks, Blake.
Assignee | ||
Comment 3•13 years ago
|
||
This is 100% guesswork on the Mac theme, but maybe it'll turn out ok! :)
Assignee: nobody → squibblyflabbetydoo
Attachment #517987 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #521125 -
Flags: feedback?(bwinton)
Assignee | ||
Comment 4•13 years ago
|
||
If that version looks awful, then a screenshot of this page would be useful: http://www.teamboxel.com/misc/colors.html (it's a collection of swatches of all the Mozilla theme-dependent color values from https://developer.mozilla.org/en/CSS/color_value).
Status: ASSIGNED → NEW
Comment 5•13 years ago
|
||
The screenshot of the colour page: http://dl.dropbox.com/u/2301433/MacMozillaColours.png More screenshots coming soon. ;) Thanks, Blake.
Status: NEW → ASSIGNED
Comment 6•13 years ago
|
||
It looks a little too white to me, given the colour of the folder pane beside it. "WindowFrame" seems like it might be a better choice. On the other hand, the content pane is white, and the message list alternates between white and blue… Maybe it's not so bad. Bryan, Andreas, what do you two think? Thanks, Blake.
Attachment #521169 -
Flags: ui-review?(clarkbw)
Attachment #521169 -
Flags: feedback?(nisses.mail)
Comment 7•13 years ago
|
||
Comment on attachment 521125 [details] [diff] [review] Add possible Mac theme >+++ b/mail/themes/gnomestripe/mail/accountCentral.css >@@ -59,55 +60,58 @@ separator.thin { > .acctCentralLinkText:hover:active { >- color: #000000; >+ color: -moz-FieldText; > } >+++ b/mail/themes/pinstripe/mail/accountCentral.css >@@ -60,65 +60,63 @@ separator.thin { > /* ::::: links ::::: */ > .acctCentralLinkText:hover:active { >- color: #000000; >+ color: -moz-FieldText; > } I think we should use -moz-activehyperlinktext for these. I also think that the selector should be changed to ".acctCentralLinkText:active", cause I noticed some weirdness when I clicked and held down one link, and then hovered over a different one… I think you can also consider this an r=me, but feel free to re-request review if you change a bunch of stuff. Thanks, Blake.
Attachment #521125 -
Flags: review+
Attachment #521125 -
Flags: feedback?(bwinton)
Attachment #521125 -
Flags: feedback+
Comment 8•13 years ago
|
||
Comment on attachment 521169 [details]
Old and new Mac styling.
I think the white color looks pretty nice and fresh.
It also looks nice next to the calendars today pane and other white areas we use in Thunderbird.
Attachment #521169 -
Flags: feedback?(nisses.mail) → feedback+
Comment 9•13 years ago
|
||
Comment on attachment 521169 [details]
Old and new Mac styling.
white is the new gray!
Attachment #521169 -
Flags: ui-review?(clarkbw) → ui-review+
Assignee | ||
Comment 10•13 years ago
|
||
Checked in with comment 7 addressed: http://hg.mozilla.org/comm-central/rev/586c2360de48
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.3a4
You need to log in
before you can comment on or make changes to this bug.
Description
•