advanced settings in firefox developer use system's font color

RESOLVED FIXED in Firefox 36

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: vanadar.univers, Assigned: Gijs)

Tracking

35 Branch
Firefox 36
x86
Linux
Points:
1
Dependency tree / graph
Bug Flags:
firefox-backlog +
in-testsuite -
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Reporter

Description

5 years ago
User Agent: Mozilla/5.0 (X11; Linux i686; rv:35.0) Gecko/20100101 Firefox/35.0
Build ID: 20141111004002

Steps to reproduce:

I use a dark theme on KDE which set the font color to white.

i download the new Firefox Developer Edition, start it, then go to about:preferences#advanced


Actual results:

the text is in white which makes it very hard to read (see firefox-developers-advancedSettings.png)


Expected results:

the text should have been black (or the page should not set a background) like in about:preferences#security

Updated

5 years ago
Component: Untriaged → Preferences
Assignee

Comment 1

5 years ago
Note to self: use Adwaita-dark from http://askubuntu.com/questions/454372/dark-gtk-theme-for-ubuntu-14-04 to reproduce (the global dark theme and various other things I'd already installed didn't do the trick).

Jared, I don't want to add stuff to ship-in-content-prefs right now because you just triaged it, but I do think we should fix this before shipping...

In part because I can't figure out why this is happening, fwiw - the checkboxes in the security and advanced pane ought to be pretty similar, and I don't understand why the color isn't being applied in the same way. Maybe it's the tabpanel/tabbox, or something? :-\
Status: UNCONFIRMED → NEW
Component: Preferences → Theme
Ever confirmed: true
Flags: needinfo?(jaws)
Assignee

Comment 2

5 years ago
Hah, sometimes writing it down makes everything clear. What's different between advanced prefs and all the other prefs? That's right, the tabpanels. Which get their color re-set on both Windows and Linux (so this is probably reproducible on dark high contrast themes on Windows, although I've not tested this). Fix is trivial, namely copy the color declaration there, too. Tested with style editor on my Linux VM.
Attachment #8522177 - Flags: review?(jaws)
Assignee

Updated

5 years ago
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Assignee

Comment 4

5 years ago
Marco, can you add this? :-)
Flags: needinfo?(mmucci)
Comment on attachment 8522177 [details] [diff] [review]
tabpanel style should also override color because windows/linux re-set the color to -moz-dialogtext there,

>--- a/toolkit/themes/shared/in-content/common.inc.css
>+++ b/toolkit/themes/shared/in-content/common.inc.css
>@@ -70,20 +70,21 @@ xul|groupbox xul|label {
> 
> /* tabpanels and tabs */
> 
> xul|tabpanels {
>   -moz-appearance: none;
>   font-size: 1.25rem;
>   line-height: 22px;
>   border: none;
>   padding: 0;
>   background-color: transparent;
>+  color: #424e5a;

color: inherit;

r=me with that fixed
Attachment #8522177 - Flags: review?(jaws) → review+
Added to IT 36.3
Flags: needinfo?(mmucci)
Assignee

Comment 7

5 years ago
w/ color: inherit;

remote:   https://hg.mozilla.org/integration/fx-team/rev/20b670ffa014
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/20b670ffa014
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 36
You need to log in before you can comment on or make changes to this bug.