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)

Product:
Core
Component:
Layout: Tables
Platform:
x86
All
Type:
defect

Tracking

()

Status:
VERIFIED FIXED
Milestone:
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)

test case, attached
860 bytes, text/html
Details
testcase to show that cell padding is working
351 bytes, text/html
Details 
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.
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 ago → 23 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

Creator:
Created:
Updated:
Size: