57697 - style sheet order dependency for body attributes

Status: VERIFIED INVALID

(Core :: CSS Parsing and Computation, defect, P3)

Description

[pohl]

I've been playing with style sheets and have discovered what I believe to be a
bug in mozilla.  I have two style sheets, test1.css and test2.css, which are
identical except for a reversal of two lines:

test1.css:
....
body { background-color: #e8e5e0; }
div.masthead { background-color:, #e8e5e0; padding: 10px; }
...


test2.css:
...
div.masthead { background-color:, #e8e5e0; padding: 10px; }
body { background-color: #e8e5e0; }
...

For files that use test1.css, the background-color attribute appears
to be ignored by mozilla.  test2.css, on the other hand, works just fine.

Here are a couple of sample html files

http://screaming.org/oops/666.html
http://screaming.org/oops/555.html

Comment 1

[David Chester]

Confirmed with build 10236 on linux - test cases demonstrate missing background

Comment 2

[David Chester]

Confirmed on the MN6 branch with build 20001023 on linux - test cases
demonstrate missing background

Comment 3

[Asa Dotzler [:asa]]

Also seeing the problem demonstrated in the testcases with win32 trunk build
102304.  this is branch and trunk.

Comment 4

[Boris Zbarsky [:bzbarsky]]

The style sheets contain

<STYLE>
<!--

at the beginning.  Since they are linked as external stylesheets, that is parsed
as part of the first CSS directive.  As a result, that directive has invalid
syntax and is dropped.

Remove those two lines and the

-->
</STYLE>

at the end of the stylesheets and things will work as they should.

Comment 5

[pohl]

You're right, Boris.  These styles were formerly within a document, and I cut the
whole section out.  including the STYLE and comment tags.

I made a test3.css and it all works as advertised:
http://screaming.org/oops/777.html

Sorry to take up bandwidth with this.

Comment 6

[Boris Zbarsky [:bzbarsky]]

You should probably change the bug's status to "Resolved" with a resolution of
"Invalid" or something along those lines....  :)

Comment 7

[pohl]

ACK

Comment 8

[Hixie (not reading bugmail)]

Netscape's standard compliance QA team reorganised itself once again, so taking 
remaining non-tables style bugs. Sorry about the spam. I tried to get this done 
directly at the database level, but apparently that is "not easy because of the 
shadow db", "plus it screws up the audit trail", so no can do...

Comment 9

[Hixie (not reading bugmail)]

Boris is DA MAN!
VERIFIED.