There are a number of problems on this page: 1) Borders on TBODY are rendered on the wrong side of the top row. I don't know what would happen if one of the tables were 3 rows. 2) Borders on empty cells have problems in the collapsed border model. You should render them normally. 3) The fifth table (of class=four) has the upper right corner of the border missing.

Also, the right-margin on the first 2 tables (if you have the size right so that it ends up that way) is too small on initial load, but correct on reflow. Thus I'm cc:ing troy@netscape.com , since I think he's handling scrollbar stuff.

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

moving to M5

Moving to M6.

Moving to M8.

Moving to M9.

Fixing problem (2) above should be easy. You just need to change if (NS_STYLE_TABLE_EMPTY_CELLS_HIDE==cellTableStyle->mEmptyCells) renderBorder=PR_FALSE; to if ((NS_STYLE_TABLE_EMPTY_CELLS_HIDE==cellTableStyle->mEmptyCells) && (NS_STYLE_BORDER_SEPARATE!=tableStyle->mBorderCollapse)) renderBorder=PR_FALSE; (or something like that) in nsTableCellFrame::Paint.

Ack... that != I wrote should be an ==. I started looking at the "Collapse" on the right and decided to change it to a "!=", but I should have been looking at the constant on the left...

Problem (1) has two parts. What is happening is that the bottom border of the rowgroup is being put on the bottom of every cell in the rowgroup. This is because the nsTableFrame::compute*BorderForEdgeAt functions were never finished. Problems in them include: * not checking to see if at the edge of a rowgroup/colgroup (marked with "XXX") * not getting the top/left borders of the row/col or rowgroup/colgroup to the bottom or right of the current cell (not noted in source) The second problem is the disappearance of the top border.

A further test case that shows how many problems there really are: http://www.fas.harvard.edu/~dbaron/css/test/sec1706c

Marking [TESTCASE].

Moving to M10

Adding [CSS 2] to summary and moving to M12.

In the above patch, I just realized useInsets=PR_FALSE should be moved, but it doesn't seem to do anything anyway. There are still problems with the patch for attachment 966 [details], I think.

mass move to m14.

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...

Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!

Moving to M16.

Moving to M17.

Adding dependency on bug 41262 (for most of this multipart bug).

Anynews? The URL seems to rendered correctly now (2000090808).

adding some keyword nomination

Marking mozilla1.0 (hopefully sooner).

QA contact update

Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 (you can query for this string to delete spam or retrieve the list of bugs I've moved)

Fixed by meta bug 41262.

All the testcases works fine. Marking verified. Build ID: 2002030403