22246 - [blocker]Changing COLSPAN or ROWSPAN attribute in a cell doesn't trigger redraw of table

Status: VERIFIED FIXED

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

Description

[Charles Manske]

I'm not sure if this is layout or DOM - I'll let Troy decide.
Load the attached test file into viewer or apprunner.
Click on the buttons to change the COLSPAN or ROWSPAN of Cell 2 in the table.
The table doesn't redraw.
Other buttons included in the test file confirm other attributes are working
better, although setting the body background color still doesn't redraw the
the entire view correctly (this is covered by bug 1886).

Comment 1

[Charles Manske]

Attachment: Test changing COLSPAN, ROWSPAN attribute on a table cell

Comment 2

[troy]

Changing component to HTML Tables

Comment 3

[Charles Manske]

Marking as blocker for beta table editing work and adding Steve to CC.

Comment 4

[buster]

charley also sez:
Note that this is a regression - it worked when I did initial table editing. I
think it crept in sometime after 12/10.

Comment 5

[Charles Manske]

Attachment: Sample table for testing row and column insert with COLSPAN and ROWSPAN > 1

Comment 6

[Charles Manske]

Using the 12/22/99 attachment, here is an example of bad table layout as a
result of not responding the the changed rowspan:
1. Edit the 12/22 sample file
2. Place caret in cell containing "7"
3. Execute menu command Table | Insert | Row Below
See the bad table layout!
Execute Debug | Test Table Layout to show debug information: notice that the
cell with the "6" did have its rowspan changed to 4, as it should.
Note also that you can click in various cells in the table and type or resize
the window, but none of these actions cause the table to redisplay correctly.
If you us File | Save As to save the file locally, then reload that file,
the table will display correctly.

You can also test inserting a "column before" by placing caret in the cell with
"7" and use Table | Insert | Column before.
The cell with the "2" increases its COLSPAN, but again the table doesn't
redraw correctly.

Comment 7

[karnaze (gone)]

Fixed with latest checkin.

Comment 8

[karnaze (gone)]

I fixed the 1st attachment but didn't fix the 2nd. Reopening.

Comment 9

[leger]

Clearing FIXED resolution due to reopen.

Comment 10

[karnaze (gone)]

The 2nd attachment was fixed with 1/15 changes.

Comment 11

[Chris Petersen]

Fixed in the Feb 22 build.

Comment 12

[karnaze (gone)]

rowspan/colspan=0 now only works according to HTML 4 in strict mode. There were 
pages relying on the behavior where a value <= 1 became 1.