Closed Bug 579739 Opened 14 years ago Closed 14 years ago

Make lightweight themes / personas work in composer windows

Categories

(SeaMonkey :: Themes, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b2

People

(Reporter: kairo, Assigned: InvisibleSmiley)

References

Details

Attachments

(1 file, 3 obsolete files)

We should also make lwthemes work in composer windows once they work in browser.
The minimum is to add these two lines to the <window> of the XUL, cf. bug 579738:
        lightweightthemes="true"
        lightweightthemesfooter="status-bar"
Version: unspecified → Trunk
Attached patch basic lwthemes support (obsolete) — Splinter Review
I think in the case of Composer, styling the bottom tabs with text shadow is OK, so I currently see no reason to add specific styles.
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #487572 - Flags: review?(neil)
(In reply to comment #2)
> I think in the case of Composer, styling the bottom tabs with text shadow is
> OK, so I currently see no reason to add specific styles.
But the background of the bottom tabs becomes the lightweight theme solid background colour, which isn't so helpful. (It would be nice if it could be merged with the status bar, but unfortunately the sidebar gets in the way.)
Attached patch basic lwthemes support v1a (obsolete) — Splinter Review
OK, so no styling for the tabs. Also found that the formatting menu should have some transparency for the sake of consistency.
Attachment #487572 - Attachment is obsolete: true
Attachment #487878 - Flags: superreview?(neil)
Attachment #487878 - Flags: review?(neil)
Attachment #487572 - Flags: review?(neil)
Comment on attachment 487878 [details] [diff] [review]
basic lwthemes support v1a

>+#EditModeTabs:-moz-lwtheme {
I don't think this is the correct element; there's still a bit of custom colour leaking through. Try the EditModeToolbar instead.

>+  -moz-appearance: none;
Don't think we need this.
Note that there is an editor.css in classic/mac...
Attached patch basic lwthemes support v1b (obsolete) — Splinter Review
Attachment #487878 - Attachment is obsolete: true
Attachment #488015 - Flags: superreview?(neil)
Attachment #488015 - Flags: review?(stefanh)
Attachment #487878 - Flags: superreview?(neil)
Attachment #487878 - Flags: review?(neil)
Attachment #488015 - Flags: superreview?(neil) → superreview+
Comment on attachment 488015 [details] [diff] [review]
basic lwthemes support v1b

I can't seem to get Composer working in my trunk build, it just hangs (and it's not this patch that causes it of course). I do manage to get a view of the lwtheming, though.

+toolbar menulist:-moz-lwtheme /* formatting menu */ {
+  opacity: .8;
+}

Looks like you could put this in toolbar.css (also for mac ;-) ). I might re-visit the mac part since I'm undecidable on the opacity here and atm I can't test the color/bg-color thing.

Thanks!
Attachment #488015 - Flags: review?(stefanh) → review+
(In reply to comment #8)
> +toolbar menulist:-moz-lwtheme /* formatting menu */ {
> +  opacity: .8;
> +}
> 
> Looks like you could put this in toolbar.css (also for mac ;-) ).

Good idea; that also obsoletes part of the MailNews patch. :-)

> I might re-visit the mac part since I'm undecidable on the opacity here
> and atm I can't test the color/bg-color thing.

After the address book patch review I thought I might just as well remove the color definitions for Mac here, too. I mean, since you cannot test it and will have to revisit that anyway... If you disagree, speak up (r-).
Attachment #488015 - Attachment is obsolete: true
Attachment #488369 - Flags: superreview?(neil)
Attachment #488369 - Flags: review?(stefanh)
(In reply to comment #9)
> (In reply to comment #8)
> > +toolbar menulist:-moz-lwtheme /* formatting menu */ {
> > +  opacity: .8;
> > +}
> > Looks like you could put this in toolbar.css (also for mac ;-) ).
> Good idea; that also obsoletes part of the MailNews patch. :-)
What about the compose window, which has menulists in the addressing widget?
(In reply to comment #10)
> What about the compose window, which has menulists in the addressing widget?

If and when (!) this becomes an issue there we can fix it there (i.e. write a rule specific to that). I think doing the general case with a general rule and specific cases with specific rules is the way to go, and the compose window one might actually be the only exception, while we have at least two places that make use of the general rule.
Comment on attachment 488369 [details] [diff] [review]
basic lwthemes support v1c [Checkin: comment 13]

I'm pretty sure the color is needed, so it's probably better to leave both color rules in.
Attachment #488369 - Flags: review?(stefanh) → review+
Attachment #488369 - Flags: superreview?(neil) → superreview+
Comment on attachment 488369 [details] [diff] [review]
basic lwthemes support v1c [Checkin: comment 13]

(In reply to comment #12)
> I'm pretty sure the color is needed, so it's probably better to leave both
> color rules in.

Fixed upon checkin: http://hg.mozilla.org/comm-central/rev/c60c611cea9a
Attachment #488369 - Attachment description: basic lwthemes support v1c → basic lwthemes support v1c [Checkin: comment 13]
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
You need to log in before you can comment on or make changes to this bug.