Bug 13793 – RULES=groups overridden by border, colgroup=span not honored by border collapsing

• Component: Tables
• Status: RESOLVED FIXED

[Brady Eidson]

Look at the url http://www.htmlcodetutorial.com/tables/_TABLE_RULES_GROUPS.html in both ToT WebKit and Firefox, scroll to the bottom.

There is a discussion about colgroup with span.  Firefox supports it properly, but ToT WebKit puts a border between each column instead of the specified groups

[Brady Eidson]

Reduction taken from that page - 
<TABLE BORDER=1 CELLPADDING=4 RULES=GROUPS  FRAME=BOX>

<COLGROUP></COLGROUP>
<COLGROUP SPAN=3></COLGROUP>

<TR> <TH>Weekday</TH> <TH>Date</TH>    <TH>Manager</TH> <TH>Qty</TH> </TR>
<TR> <TD>Monday</TD> <TD>09/11</TD>    <TD>Kelsey</TD> <TD>639</TD>  </TR>
<TR> <TD>Tuesday</TD> <TD>09/12</TD>   <TD>Lindsey</TD> <TD>596</TD> </TR>
<TR> <TD>Wednesday</TD> <TD>09/13</TD> <TD>Randy</TD> <TD>1135</TD>  </TR>
<TR> <TD>Thursday</TD> <TD>09/14</TD>  <TD>Susan</TD> <TD>1002</TD>  </TR>
<TR> <TD>Friday</TD> <TD>09/15</TD>    <TD>Randy</TD> <TD>908</TD>   </TR>
<TR> <TD>Saturday</TD> <TD>09/16</TD>  <TD>Lindsey</TD> <TD>371</TD> </TR>
<TR> <TD>Sunday</TD> <TD>09/17</TD>    <TD>Susan</TD> <TD>272</TD>   </TR>
</TABLE>

[Dave Hyatt]

I assume this is incorrect but will need to verify.  The bug is that border= on table is causing every cell to get a border, without respect for the rules attribute.

Another thing that I notice here is that the default colgroup span in WebKit is 0, whereas in the spec and in Firefox it is 1

[Dave Hyatt]

There are two bugs here.  RULES=groups is improperly being ignored by border= on a table.  colgroup span is not honored when doing border collapsing.

[Dave Hyatt]

Created attachment 14639 [details]
Patch to fix first issue.

Two of the Mozilla tables tests in the expected failures directory now pass with this bug fix.  No other tests are affected.

Comment on attachment 14639 [details]
Patch to fix first issue.

r=me

[Dave Hyatt]

Created attachment 14640 [details]
Patch to fix second issue

Comment on attachment 14640 [details]
Patch to fix second issue

r=me. you can remove the {start,end}ColEdge = false; lines

[Dave Hyatt]

Fixed.

[David Kilzer (:ddkilzer)]

(In reply to comment #5)
> Created an attachment (id=14639) [edit]
> Patch to fix first issue.
> 
> Two of the Mozilla tables tests in the expected failures directory now pass
> with this bug fix.  No other tests are affected.

Shouldn't these tests be svn-moved to the non-failing directory if they now pass?

[David Kilzer (:ddkilzer)]

(In reply to comment #9)
> Fixed.

Code: r21614, r21615
Tests: r21616, r21617

[David Kilzer (:ddkilzer)]

(In reply to comment #10)
> (In reply to comment #5)
> > Two of the Mozilla tables tests in the expected failures directory now pass
> > with this bug fix.  No other tests are affected.
> Shouldn't these tests be svn-moved to the non-failing directory if they now
> pass?

Actually three tests now pass when compared to Firefox 2.0.0.3 and 3.0a5pre (from 27-May-2007 5:24 AM):

LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct.html
LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px.html
LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct.xml

[David Kilzer (:ddkilzer)]

$ svn commit LayoutTestResults LayoutTests/ChangeLog LayoutTests/tables
Sending        LayoutTestResults/ChangeLog
Adding         LayoutTestResults/qt/tables/mozilla/marvin/colgroup_width_pct-expected.txt
Adding         LayoutTestResults/qt/tables/mozilla/marvin/colgroup_width_px-expected.txt
Adding         LayoutTestResults/qt/tables/mozilla/marvin/x_colgroup_width_pct-expected.txt
Deleting       LayoutTestResults/qt/tables/mozilla_expected_failures/marvin/colgroup_width_pct-expected.txt
Deleting       LayoutTestResults/qt/tables/mozilla_expected_failures/marvin/colgroup_width_px-expected.txt
Deleting       LayoutTestResults/qt/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.txt
Sending        LayoutTests/ChangeLog
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_pct-expected.checksum
Adding  (bin)  LayoutTests/tables/mozilla/marvin/colgroup_width_pct-expected.png
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_pct-expected.txt
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_pct.html
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_px-expected.checksum
Adding  (bin)  LayoutTests/tables/mozilla/marvin/colgroup_width_px-expected.png
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_px-expected.txt
Adding         LayoutTests/tables/mozilla/marvin/colgroup_width_px.html
Adding         LayoutTests/tables/mozilla/marvin/x_colgroup_width_pct-expected.checksum
Adding  (bin)  LayoutTests/tables/mozilla/marvin/x_colgroup_width_pct-expected.png
Adding         LayoutTests/tables/mozilla/marvin/x_colgroup_width_pct-expected.txt
Adding         LayoutTests/tables/mozilla/marvin/x_colgroup_width_pct.xml
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct-expected.checksum
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct-expected.png
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct-expected.txt
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_pct.html
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px-expected.checksum
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px-expected.png
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px-expected.txt
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/colgroup_width_px.html
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.checksum
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.png
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.txt
Deleting       LayoutTests/tables/mozilla_expected_failures/marvin/x_colgroup_width_pct.xml
Transmitting file data ..
Committed revision 21835.