185672 - colspan ="0" should stop at colgroup boundaries

Status: NEW

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

Description

[Bernd]

quote from html 4.0.1

This attribute specifies the number of columns spanned by the current cell. The
default value of this attribute is one ("1"). The value zero ("0") means that
the cell spans all columns from the current column to the last column of the
column group (COLGROUP) in which the cell is defined.


http://lxr.mozilla.org/seamonkey/source/layout/html/table/src/nsCellMap.cpp#2138
seems to be wrong

 PRInt32 endColIndex = (0 == cell->GetColSpan()) ? numColsInTable - 1 :
aColIndex + cellColSpan - 1;

Comment 1

[Bernd]

Attachment: testcase

Comment 2

[Grzegorz Adam Hankiewicz]

Attachment: Test case without colgroup fails too.

Also wrong when no colgroup is specified. The default colgroup spans the whole
table.

Comment 3

[Bernd]

Comment on attachment 130484 [details]
Test case without colgroup fails too.

the testcase is invalid, 0 col and rowspans dont work in quirks mode. Use a
strict doctype and it will work

Comment 4

[Oskar Liljeblad]

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

Comment 5

[Bernd]

This bug will change once we do what http://dev.w3.org/html5/spec/Overview.html#attr-tdth-colspan requires. To remove the 0 colspans entirely