More lwtheme adjustments

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
Themes
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: stefanh, Assigned: stefanh)

Tracking

Trunk
seamonkey2.1b2

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

2.58 KB, patch
neil@parkwaycc.co.uk
: review+
neil@parkwaycc.co.uk
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

7 years ago
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.
Attachment #490461 - Flags: superreview?(neil)
Attachment #490461 - Flags: review?(neil)

Comment 1

7 years ago
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.]
Attachment #490461 - Flags: superreview?(neil)
Attachment #490461 - Flags: superreview+
Attachment #490461 - Flags: review?(neil)
Attachment #490461 - Flags: review+
(Assignee)

Comment 2

7 years ago
http://hg.mozilla.org/comm-central/rev/baef808c1634
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.