107857 - WRMB:[quirks]Show Empty Table Cells STYLE background-color, and background-image

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect, P2)

Description

[Bob Clary [:bc] (inactive)]

bug 94041 resulted in the background colors/images displaying in Quirks mode for
empty TDs with BGCOLOR/BACKGROUND attributes.

It seems appropriate to make the solution complete so that in Quirks mode
background colors or background images for table cells will show for empty cells
whether you code the BGCOLOR/BACKGROUND attributes on the cell or you code the
background-color, background-image via CSS. This will duplicate the behavior of
IE 5.5.

Comment 1

[Bob Clary [:bc] (inactive)]

Attachment: BGCOLOR/BACKGROUND example

Comment 2

[Bob Clary [:bc] (inactive)]

Attachment: background-color/background-image example

Comment 3

[rubydoo123]

adding WRMB bugscape transition notations

Comment 4

[karnaze (gone)]

Attachment: change to quirk.css to always show backgrouds on empty cells in quirks mode

Comment 5

[Alexandru Savulov]

Comment on attachment 56218 [details] [diff] [review]
change to quirk.css to always show backgrouds on empty cells in quirks mode

r= alexsavulov

Comment 6

[Bernd]

Chris, I think it would be good to get an additional 'r' from Hixie (the gray
eminence of *.css)

Comment 7

[karnaze (gone)]

Attachment: revised patch

Comment 8

[Marc Attinasi]

Comment on attachment 56255 [details] [diff] [review]
revised patch

sr=attinasi

Comment 9

[Judson Valeski]

what sites are affected by this?

Comment 10

[Bob Clary [:bc] (inactive)]

I do not know of any sites that are affected by this. It came out of documenting
the compatibility fix of bug 94041. It seemed inconsistent to tell developers
that bgcolor and background attributes would work on TD in quirks mode but
background-color and background-image CSS rules would not. If this is a big
issue, then mark it wontfix.

Comment 11

[karnaze (gone)]

-->trunk.

Comment 12

[Markus Gerstel]

Does this patch cause empty cells (for example on the tinderbox pages) to be
displayed ?

Comment 13

[karnaze (gone)]

The backgrounds will be displayed like IE does. Is this a problem?

Comment 14

[Markus Gerstel]

Attachment: I didn't mean the background but the border

Comment 15

[karnaze (gone)]

Yes, empty-cells applies to borders too.

Comment 16

[Chris Waterson]

Hey karnaze, we now display tinderbox differently than IE (e.g.). Was this patch
too violent?

Comment 17

[karnaze (gone)]

Yea, I should have realized that a bug this easy has got to be wrong somehow. 
After to talking to attinasi, I need to teach the css parser to recognize a 
value like -moz-show-background-hide-borders on empty-cells.

Comment 18

[karnaze (gone)]

Attachment: patch to add -moz-show-background value to empty-cells and make that the quirk default

Comment 19

[Pierre Saslawsky]

Comment on attachment 56768 [details] [diff] [review]
patch to add -moz-show-background value to empty-cells and make that the quirk default

r=pierre

Comment 20

[Marc Attinasi]

Comment on attachment 56768 [details] [diff] [review]
patch to add -moz-show-background value to empty-cells and make that the quirk default

sr=attinasi - nice non-violent patch

Comment 21

[karnaze (gone)]

The new patch is in.

Comment 22

[karnaze (gone)]

*** Bug 108730 has been marked as a duplicate of this bug. ***

Comment 23

[karnaze (gone)]

*** Bug 108896 has been marked as a duplicate of this bug. ***