Last Comment Bug 612144 - More lwtheme adjustments
: More lwtheme adjustments
Status: RESOLVED FIXED
:
Product: SeaMonkey
Classification: Client Software
Component: Themes (show other bugs)
: Trunk
: All All
: -- normal (vote)
: seamonkey2.1b2
Assigned To: Stefan [:stefanh] (away until May 28)
:
Mentors:
Depends on:
Blocks: 579738
  Show dependency treegraph
 
Reported: 2010-11-14 10:04 PST by Stefan [:stefanh] (away until May 28)
Modified: 2010-11-14 12:26 PST (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Fix more lwtheme stuff (2.58 KB, patch)
2010-11-14 10:04 PST, Stefan [:stefanh] (away until May 28)
neil: review+
neil: superreview+
Details | Diff | Review

Description Stefan [:stefanh] (away until May 28) 2010-11-14 10:04:31 PST
Created attachment 490461 [details] [diff] [review]
Fix more lwtheme stuff

I think we can put the treecols rule in communicator, as I don't need/want the color styling (the colors are in tree.css and I also set some colors on the vboxes). I think it should work for non-mac, but I haven't tested. I put back the colors in the mac version of addressbook.css as I discovered that I need the color and I believe (even though it doesn't have any affect) the bg-color should be there.
Comment 1 neil@parkwaycc.co.uk 2010-11-14 11:37:45 PST
Comment on attachment 490461 [details] [diff] [review]
Fix more lwtheme stuff

>+#msgHeaderView:-moz-lwtheme {
>   text-shadow: none;
>   color: -moz-dialogtext;
>   background-color: -moz-dialog;
> }
[I get the feeling that we should have a class for this so that we can simply mark the bits of "normal" UI as non-themed.]
Comment 2 Stefan [:stefanh] (away until May 28) 2010-11-14 12:26:58 PST
http://hg.mozilla.org/comm-central/rev/baef808c1634

Note You need to log in before you can comment on or make changes to this bug.