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?).

Tbh, it doesn't make sense to reject this proposal since you can already change the toolbar background. If you have a black toolbar background, you should be able to change the corresponding text color to white.

Do we do this in a theme that we're shipping with the browser?

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #2) > Do we do this in a theme that we're shipping with the browser? Yes, the default theme (on macOS at least) has a tabbar with a dark background (and white text), and a navbar with a light background (and black text). Atm, the theming API doesn't allow implementing the default theme, the closest you can get is: { colors: { toolbar: "#f9f9fa", textcolor: "#000", accentcolor: "#0c0c0d" } } With that you would get a dark tab bar, but the text in the tab bar would be black. You could also change textcolor to be white, but then the navbar would have white on light gray text. Which is why there should be a way of modifying one text color independently from the other.

Comment on attachment 8902658 [details] Bug 1387582 - Add toolbar_text color property to theming API. https://reviewboard.mozilla.org/r/174330/#review181330 API parts look good to me! ::: browser/themes/linux/browser.css:61 (Diff revision 2) > } > > #navigator-toolbox > toolbar:not(#toolbar-menubar):not(#TabsToolbar) { > background-color: var(--toolbar-bgcolor); > background-image: var(--toolbar-bgimage); > + color: var(--toolbar-color, inherit); I'm not 100% sure that this does what we want/ need wrt the `inherit` statement (since it rarely has in my experience), so it's good that you flagged Dão to look at this.

Hi Dão, I would like to land this in the 57 cycle as it is a blocker for my extension. Do you think you could review this soon?

Comment on attachment 8902658 [details] Bug 1387582 - Add toolbar_text color property to theming API. https://reviewboard.mozilla.org/r/174330/#review185430

Thank you!

Pushed by ntim.bugs@gmail.com: https://hg.mozilla.org/integration/autoland/rev/7bda9cc7b798 Add toolbar_text color property to theming API. r=dao,mikedeboer

I've updated https://developer.mozilla.org/en-US/Add-ons/WebExtensions/manifest.json/theme with these new properties, but let me know if we need anything else.

Looks mostly good to me. Here are a few comments: - "the background color for the toolbar." The color applies to multiple toolbars (the navigation bar, the bookmarks bar, and the selected tab). - I think it would be helpful to have a small sketch of where colors apply

I can reproduce this issue on Firefox 57.0a1 (20170803134456) under Wind 10 64-bit. This issue is verified as fixed on Firefox 57.0b11 (20171023180840) under Wind 10 64-bit and Mac OS X 10.13. See attached gif.

> - I think it would be helpful to have a small sketch of where colors apply I've made an example with a screenshot. I hope this covers it well enough: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/manifest.json/theme#example-screenshot

(In reply to Will Bamberg [:wbamberg] from comment #15) > > - I think it would be helpful to have a small sketch of where colors apply > > I've made an example with a screenshot. I hope this covers it well enough: > > https://developer.mozilla.org/en-US/Add-ons/WebExtensions/manifest.json/ > theme#example-screenshot Very useful, thanks!