The default bug view has changed. See this FAQ.

Make lightweight themes / personas work in composer windows

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
Themes
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Robert Kaiser, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.1b2

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Reporter)

Description

7 years ago
We should also make lwthemes work in composer windows once they work in browser.
(Assignee)

Comment 1

7 years ago
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
(Assignee)

Comment 2

7 years ago
Created attachment 487572 [details] [diff] [review]
basic lwthemes support

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)

Comment 3

7 years ago
(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.)
(Assignee)

Comment 4

7 years ago
Created attachment 487878 [details] [diff] [review]
basic lwthemes support v1a

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 5

7 years ago
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.

Comment 6

7 years ago
Note that there is an editor.css in classic/mac...
(Assignee)

Comment 7

7 years ago
Created attachment 488015 [details] [diff] [review]
basic lwthemes support v1b
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)

Updated

7 years ago
Attachment #488015 - Flags: superreview?(neil) → superreview+

Comment 8

7 years ago
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+
(Assignee)

Comment 9

7 years ago
Created attachment 488369 [details] [diff] [review]
basic lwthemes support v1c [Checkin: comment 13]

(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?
(Assignee)

Comment 11

7 years ago
(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 12

7 years ago
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+

Updated

7 years ago
Attachment #488369 - Flags: superreview?(neil) → superreview+
(Assignee)

Comment 13

7 years ago
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]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
You need to log in before you can comment on or make changes to this bug.