1400575 - Port bug 1387582 to TB [Allow modifying the toolbar text color separately from the global text color]

Status: RESOLVED FIXED

(Thunderbird :: Theme, enhancement)

Description

[Richard Marti (:Paenglab)]

From bug 1387582: You can set the global text color (colors.textcolor) and a toolbar background (colors.toolbar), but you can't set the text color of the toolbars (colors.toolbartext?).

Comment 1

[Richard Marti (:Paenglab)]

Attachment: toolbarTextColor.patch

Direct port of https://hg.mozilla.org/mozilla-central/rev/7bda9cc7b798 (CSS part) plus added support for status bar which no more exists in FX.

Comment 2

[Richard Marti (:Paenglab)]

Attachment: nuvola-webext-theme.xpi

For easier review I add a test theme which makes the text red.

Comment 3

[Jorg K (CEST = GMT+2)]

Looks like I need to refresh by build here first to make it work :-(

Comment 4

[Jorg K (CEST = GMT+2)]

With the patch and the special test theme, the text colour on toolbars goes red. Also, the colour on the compose window's menu (File, Edit, View, etc.) and the text of the active tab in the main window. The latter is most likely due to:
	
.tabmail-tab[visuallyselected=true]:-moz-lwtheme {
  color: var(--toolbar-color, inherit);
}

Is that really desired?

Comment 5

[Richard Marti (:Paenglab)]

Attachment: toolbarTextColor.patch

(In reply to Jorg K (GMT+2) from comment #4)
> With the patch and the special test theme, the text colour on toolbars goes
> red. Also, the colour on the compose window's menu (File, Edit, View, etc.)
> and the text of the active tab in the main window. The latter is most likely
> due to:
> 	
> .tabmail-tab[visuallyselected=true]:-moz-lwtheme {
>   color: var(--toolbar-color, inherit);
> }
> 
> Is that really desired?

Yes, because normally when someone sets the text color he sets the toolbar background color too. And this background color is also on the selected tab.

I'll attach a better test theme to show this.

I found a bug there Win7 Aero's menubar hadn't the correct colors because it is special where the menubar is positioned. I also moved the addressbook.css code to the shared file.

Comment 6

[Richard Marti (:Paenglab)]

Attachment: test-webext-theme.xpi

Better test theme.

Comment 7

[Jorg K (CEST = GMT+2)]

Comment on attachment 8909099 [details] [diff] [review]
toolbarTextColor.patch

(In reply to Richard Marti (:Paenglab) from comment #6)
> Better test theme.
That theme is terrible :-(

;-)

Comment 8

[Richard Marti (:Paenglab)]

Attachment: toolbarTextColor.patch

Only change: I added a '-moz-appearance: none;' to OSX AB's and composer's toolboxes to really show the background color.

(In reply to Jorg K (GMT+2) from comment #7)
> (In reply to Richard Marti (:Paenglab) from comment #6)
> > Better test theme.
> That theme is terrible :-(

I hope you didn't got eye cancer.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/5a2d4268fbd5
Port bug 1387582 to TB [Add toolbar_text color property to theming API]. r=jorgk