475268 - CSS for XML tree view changes background color but leaves text color as default

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Mihai Capotă]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; ro; rv:1.9.0.5) Gecko/2008121621 Ubuntu/8.04 (hardy) Firefox/3.0.5
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; ro; rv:1.9.0.5) Gecko/2008121621 Ubuntu/8.04 (hardy) Firefox/3.0.5

The default style for XML rendering changes the background color but leaves text color as default.
This is a problem when the default text color is not black, e.g. when using a light text on dark background theme.

Reproducible: Always

Steps to Reproduce:
1. Set the text color to white and the background color to black.
2. Open an XML file, e.g. http://www.w3schools.com/xml/cd_catalog.xml
Actual Results:  
The background of the page is set to white, while the element content text is left as default, also white, thus unreadable.

Expected Results:  
The page should be readable. The element content text color should be set to something dark to contrast with the background.

Comment 1

[philippe (part-time)]

Indeed it is. I thought there was already a bug somewhere, but can't find it.

It is a bit similar to bug 437366...
According to the DomInspector, the background of the unstyled XML page is set to 'white' (*|*:root {} in viewsource.css) but no color is specified.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: Like so

Jonas, what do you think of also moving the -moz-fixed font-family decl?

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 358784 [details] [diff] [review]
Like so

don't know enough about -moz-fixed to have an opinion on that.

Comment 4

[Boris Zbarsky [:bzbarsky]]

moz-fixed just means the user's default monospace font.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Pushed http://hg.mozilla.org/mozilla-central/rev/a6bcf72d1d75

Comment 6

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Oh, I'm with you. Would make sense to move it, but I don't care much either way