| Swainr | |
| Jan 8 2015, 12:33 PM |
| F26989: pasted_file | |
| Jan 8 2015, 2:13 PM |
| F26977: s.PNG | |
| Jan 8 2015, 12:33 PM |
Environment- beta, test2
1> insert a table.By default it is 4x4.
2> Highlight only the content cells under the first two header and merge them. Do the same for the other two columns. It is now a 2 x 4 table.
3> Select the last header cell and insert after or click on the first header cell and insert before.
JS error “Uncaught TypeError: Cannot read property 'isPlaceholder' of null” appears.
Screenshot attached.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Remove redundant rows/cols after cell merge | VisualEditor/VisualEditor | master | +34 -4 |
The problem is that you are generating an invalid table by merging entire rows. The HTML you generate, with rowspan="3" but not actually spanning any rows will break in most browsers. Our merge tool should be able to deal with this though.
This is actually conceptually not that straightforward, consider the following merge operation:
This results in table going from 4 logical rows to 3.
Change 183510 had a related patch set uploaded (by Esanders):
Remove redundant rows/cols after cell merge