25148 - Table layout not correct after inserting row/column when colspan=0 or rowspan=0

Status: VERIFIED FIXED

(Core :: Layout, defect, P3)

Description

[Charles Manske]

Load the test file into Composer
For colspan test:
1. Put caret in "Cell 2"
2. Execute menu command: Table|Insert|Column before
A new columns is inserted, but the cell 4, with colspan=0, should now extend
one more cell to the end of the table.

For rowspan test:
1. Put caret in "Cell 6"
2. Execute Table|Insert|Row below
A new row is inserted, but cell 5, with rowspan=0, should now extend one more
cell downward to the end of the table.

Comment 1

[Charles Manske]

Attachment: Test table with colspan=0 and rowspan=0 cells

Comment 2

[karnaze (gone)]

Charlie, get my checkins from this evening and tell me if the 1st problem is 
solved. I'm not sure what the editor is doing when it inserts a column.

Also, in the 2nd problem, nsHTMLEditor::InsertTableRow gets into an infinite 
loop because it is calling GetCellDataAt with the same row and col indices.

Comment 3

[Charles Manske]

Attachment: The CORRECT table to test this bug

Comment 4

[Charles Manske]

Sorry, the table supplied was not the correct one.
The problem seems fixed for the COLSPAN=0 case.
Use the new table to test the ROWSPAN=0 case -- that problem still occurs.
I'll look into the infinite loop problem -- thanks for finding it!

Comment 5

[karnaze (gone)]

Fixed with latest checkin.

Comment 6

[Chris Petersen]

Fixed in the Feb 22 build.