Closed Bug 2130 Opened 26 years ago Closed 23 years ago

Collapsing border model and cell padding (ugly rendering)

Categories

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

x86
All
defect

Tracking

()

VERIFIED FIXED
mozilla0.9.9

People

(Reporter: troy, Assigned: karnaze)

References

Details

(Keywords: css2, testcase, Whiteboard: [awd:tbl][TESTCASE] likely dups: 12462, 22897; also 26614, 31256)

Attachments

(2 files)

The collapsing table border model seems to ignore table cell padding. The example below demonstrates that. Notice how small each table cell is. Change the border model to 'separate' and the cells are larger <html> <head> <style> TABLE { border-collapse: collapse; border: 5px solid yellow; } *#col1 { border: 3px solid black; } TD { border: 1px solid red; padding: 1em; } TD.solid-blue { border: 5px dashed blue; } TD.solid-green { border: 5px solid green; } </style> </head> <body> <P> <TABLE> <COL id="col1"><COL id="col2"><COL id="col3"> <TR id="row1"> <TD> 1 <TD> 2 <TD> 3 </TR> <TR id="row2"> <TD> 4 <TD class="solid-blue"> 5 <TD class="solid-green"> 6 </TR> <TR id="row3"> <TD> 7 <TD> 8 <TD> 9 </TR> <TR id="row4"> <TD> 10 <TD> 11 <TD> 12 </TR> <TR id="row5"> <TD> 13 <TD> 14 <TD> 15 </TR> </TABLE> </body> </html>
Setting all current Open/Normal to M4.
per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
changed misc bugs to M6
setting my table bugs to M9
Target Milestone: M9 → M11
Target Milestone: M11 → M12
Assignee: buster → karnaze
Status: ASSIGNED → NEW
Priority: P2 → P4
Target Milestone: M12 → M15
post beta, low priority table collapsing borders bugs
Status: NEW → ASSIGNED
The original report on this bug looks fixed, I think. However, the test case looks bad for other reasons. I'll attach it.
Attached file test case, attached (deleted) —
Summary: Collapsing border model and cell padding → {css2} Collapsing border model and cell padding
Keywords: css2
Migrating from {css2} to css2 keyword. The {css1}, {css2}, {css3} and {css-moz} radars should now be considered deprecated in favour of keywords. I am *really* sorry about the spam...
Whiteboard: [MAKINGTEST] bugzilla@friendshipchurchonline.org
I think I have the problem isolated even further.
Whiteboard: [MAKINGTEST] bugzilla@friendshipchurchonline.org
Sorry, I have the wrong bug. I thought that I was on a different page.
Summary: {css2} Collapsing border model and cell padding → Collapsing border model and cell padding
Summary: Collapsing border model and cell padding → Collapsing border model and cell padding (ugly rendering)
Whiteboard: likely dups: 12462, 22897; also 26614, 31256
Moving to M16.
Target Milestone: M15 → M16
Keywords: testcase
Whiteboard: likely dups: 12462, 22897; also 26614, 31256 → [TESTCASE] likely dups: 12462, 22897; also 26614, 31256
Moving to M17.
Target Milestone: M16 → M17
*** This bug has been marked as a duplicate of 41262 ***
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
I should have made a dependency instead of a dup.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Status: REOPENED → ASSIGNED
Depends on: 41262
I agree with David's comment from way back - cell padding IS working with border-collapse: collapse value. I am attaching a simplified test. Karnaze - please mark fixed if you agree.
Milestone 0.8 has beend released. We should either resolve this bug or update its milestone.
Target Milestone: M17 → ---
moving to m1.0
Target Milestone: --- → mozilla1.0
QA contact update
QA Contact: chrisd → amar
this looks fixed IMHO. anthonyd
Whiteboard: [TESTCASE] likely dups: 12462, 22897; also 26614, 31256 → [awd:tbl][TESTCASE] likely dups: 12462, 22897; also 26614, 31256
It looks fixed because collapsing borders are disabled.
Target Milestone: mozilla1.0 → mozilla0.9.7
collapsing border bugs moved to m098
Target Milestone: mozilla0.9.7 → mozilla0.9.8
->m099
Target Milestone: mozilla0.9.8 → mozilla0.9.9
OS: Windows NT → All
*** Bug 124183 has been marked as a duplicate of this bug. ***
Marking nsbeta1+
Keywords: nsbeta1+
Fixed by meta bug 41262.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago23 years ago
Resolution: --- → FIXED
All the testcases looks good. They look better than in IE. Marking verified. Build ID: 2002030403
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: