Closed Bug 3933 Opened 26 years ago Closed 25 years ago

ua.css has strange rules for tables

Categories

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

x86
Windows 98
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: ian, Assigned: peterl-retired)

References

()

Details

(Whiteboard: waiting for compat/strict ua.css files)

In ua.css, one finds the following rules for table elements: table { display: table; /* ... */ background: transparent; } "background: transparent" is redundant in ua.css, since "transparent" is the inital value. td, th { vertical-align: inherit; background: inherit; /* ... */ } "background: inherit" here will cause the table's background image to be drawn for each cell, instead of backing the table. Ditto for tr, tbody, thead and tfoot, which all have "background: inherit".
Assignee: karnaze → peterl
ua.css was recently changed so that these internal table elements only inherit background-color. This is still not 100% correct and we need 2 ua.css files, one for NavQuirks mode and one for Standard mode. The current ua.css tries to both jobs. Peter, please decide if this is still a bug.
Err, BTW, the 1999-03-23 build still has the old rules. Also, you don't want to be inherting *any* of the background, do you? If cells inherit background-color, then any background image set on tables or rows will be overwritten by the cells' backgrounds. You want *no* background rules on *any* table elements in ua.css, letting the initial value of "transparent" to do its work.
Status: NEW → ASSIGNED
Target Milestone: M6
Whiteboard: waiting for compat/strict ua.css files
ua.css, this bug, and and many other table style bugs will be easily fixed after Peter adds a new style rule (in the code) which gets applied after the html style sheet but before the css style sheets.
Target Milestone: M6 → M7
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Marking verified. These rules no longer appear in ua.css. Background issues are now different in standard and quirks mode.
You need to log in before you can comment on or make changes to this bug.