137236 - Style sheet support doesn't work in one case

Status: VERIFIED INVALID

(Core :: CSS Parsing and Computation, defect)

Description

[Brad]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.9+) Gecko/20020407
BuildID:    2002040706

I have a style sheet with the following identical definition other than the
class name:

table.updateWrapper { background: #e0e0e0; border-collapse: collapse;
border-spacing: 0pt; width: 100%; }

table.articleWrapper { background: #e0e0e0; border-collapse: collapse;
border-spacing: 0pt; width: 100%; }

When instantiating a table of class articleWrapper, I get the correct behavior,
when instantiating a table of class updateWrapper, the table has a border (it
should not).  If you need a few files to reproduce this, I can deliver them, but
I will not publish my page with the broken behavior, sorry.

Reproducible: Always
Steps to Reproduce:
1.Construct a style sheet with the above definitions
2.Build a complex html page using the table definitions
3.View the html

Actual Results:  <table class="updateWrapper"> had a border, <table
class="articleWrapper"> did not.

Expected Results:  Neither table should have a border.

email me for the pages at: [e-mail]

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Could you attach a testcase to the bug?

Comment 2

[Brad]

Attachment: Style Sheet demonstrating defs

Here is the style sheet I use, I should have pruned it, but the page I'm going
to send needs it to render worth a damn.

Comment 3

[Brad]

Attachment: Html file that uses accompanying stylesheet

Here is the html file, again unpruned.	If it absolutely has to be pruned to
the minimum bug case I can do it, but I don't know how long it will be before
I'm willing to put forth that effort, and I wanted to get the bug submitted so
that perhaps it wouldn't make it into 1.0.

Comment 4

[Brad]

To observe defective behavior just toggle the table that uses class 
updateWrapper between updateWrapper and articleWrapper and you'll see the 
rendering change.  My system is dynamically rendered so its a bit of a pain to 
construct html without a bit more effort than I truly desire to put forth, but 
keep me apprised and I'll do the best I can, assuming you tag this to 1.0.   
 
If you tag it to a later version, I'll obviously be less willing to help (i.e. 
you will have screwed me :-) ) 

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: Above HTML modified to point to stylesheet attachment

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Marking bug INVALID.  The stylesheet has a stray semicolon after the rule before
the selector "table.updateWrapper".  According to the error-handling rules in
the CSS spec, this makes the selector "; table.updateWrapper", which is an
invalid selector and therefore ignored.  Any browser that does anything with
that rule is violating the error handling rules in the CSS spec.

Comment 7

[Hixie (not reading bugmail)]

Yup: --------------------------------------------------+
                                                       |
                                                      \|/
                                                       '
   td.menuItem { background: #ffffff; color: #000080; };

   /** Update style definition */
   table.updateWrapper { background: #e0e0e0; border-collapse: collapse; 

VERIFIED INVALID