1420254 - Add additional Theme API colors to TB

Status: RESOLVED FIXED

(Thunderbird :: Theme, enhancement)

Description

[Richard Marti (:Paenglab)]

FX has added some themeAPI colors.

This are:
- toolbar_top_separator, the color of the toolbar top border and tabs border
- toolbar_bottom_separator, the color of the toolbar bottom border
- tab_text, the text color of the active tab
- toolbar_field_border, the search field border color

Comment 1

[Richard Marti (:Paenglab)]

Attachment: additional-themeAPI-colors.patch

Jörg doesn't accept reviews. Aceman, could you review it?

I'll attach a test theme for trying.

Comment 2

[Richard Marti (:Paenglab)]

Attachment: test-webext-theme.xpi

With this theme the bottom toolbar border is red, the top border blue. The active tab text is black. the search field border is white.

Don't use it too long to not become eye cancer. ;)

Comment 3

[:aceman]

(In reply to Richard Marti (:Paenglab) from comment #2)
> Created attachment 8931468 [details]
> test-webext-theme.xpi
> 
> With this theme the bottom toolbar border is red, the top border blue. The
> active tab text is black. the search field border is white.

I don't see exactly these colors.

I see no bottom toolbar border.
Top border IS blue.
Active tab text (name) IS black.
Search field border (e.g. on quick filter bar) is grey, and blue when focused.

I also see a green-pink checkerboard as background of main menu and tab bar.
Also labels of main toolbar buttons are red.
Background of main toolbar and status bar is green.

Comment 4

[Richard Marti (:Paenglab)]

Attachment: additional-themeAPI-colors.patch

(In reply to :aceman from comment #3)
> (In reply to Richard Marti (:Paenglab) from comment #2)
> > Created attachment 8931468 [details]
> > test-webext-theme.xpi
> > 
> > With this theme the bottom toolbar border is red, the top border blue. The
> > active tab text is black. the search field border is white.
> 
> I don't see exactly these colors.
> 
> I see no bottom toolbar border.
> Top border IS blue.

Great, tested on all platforms except Linux. And Linux makes it different. :( -> fixed.

> Active tab text (name) IS black.
> Search field border (e.g. on quick filter bar) is grey, and blue when
> focused.

Focused is okay when it's blue.

> I also see a green-pink checkerboard as background of main menu and tab bar.
> Also labels of main toolbar buttons are red.
> Background of main toolbar and status bar is green.

That's all correct. This test theme checks all possible themeAPI colors.

Comment 5

[:aceman]

Comment on attachment 8931618 [details] [diff] [review]
additional-themeAPI-colors.patch

Review of attachment 8931618 [details] [diff] [review]:
-----------------------------------------------------------------

OK, thanks, now I also see red bottom border of main toolbar.

Comment 6

[Richard Marti (:Paenglab)]

Thanks. :)

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/e716b8199c2d
Add additional Theme API colors to TB. r=aceman