Opened 16 years ago
Last modified 13 years ago
#2859 confirmed Bug
Wrong rowspan and empty rows after merging down rowspanned cells
Reported by: | Koen Willems | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Core : Tables | Version: | 3.0 |
Keywords: | Cc: | kwillems@… |
Description
Take a table like this:
<table title="table" cellspacing="0" cellpadding="0" summary="table"> <thead> <tr> <th scope="col">1</th> <th scope="col">2</th> </tr> </thead> <tbody> <tr> <td>a</td> <td rowspan="3">b</td> </tr> <tr> <td>c</td> </tr> <tr> <td>d</td> </tr> <tr> <td>e</td> <td>f</td> </tr> </tbody> </table>
Now merge down cell c with cell d and switch to SourceMode to look at the code. Cell b still has a rowspan 3 and cell c has a rowspan 2. Furthermore an empty row appears.
Attachments (2)
Change History (12)
comment:1 Changed 16 years ago by
Changed 16 years ago by
Attachment: | 2859.patch added |
---|
comment:2 Changed 16 years ago by
Component: | General → Core : Output Data |
---|---|
Keywords: | Confirmed added |
Version: | FCKeditor 2.6.4 → FCKeditor 2.3.2 |
comment:3 Changed 16 years ago by
Cc: | kwillems@… added |
---|
comment:4 Changed 15 years ago by
Owner: | set to Martin Kou |
---|---|
Status: | new → assigned |
Changed 15 years ago by
Attachment: | 2859_2.patch added |
---|
comment:5 Changed 15 years ago by
Keywords: | Review? added |
---|
The proposed patch includes the changes in #2813.
comment:6 Changed 15 years ago by
Milestone: | FCKeditor 2.6.5 |
---|
There are too many changes being proposed to the code. As the 2.6.5 is mainly concerned about compatibility, it's better to avoid the risk of braking this.
We may include these changes in a future version, probably on CKEditor instead.
comment:7 Changed 14 years ago by
Component: | Core : Output Data → Core : Tables |
---|
comment:8 Changed 13 years ago by
Keywords: | Confirmed removed |
---|---|
Version: | FCKeditor 2.3.2 → 3.0 |
This issue is also reproducible in latest CKEditor 3.6.2
comment:9 Changed 13 years ago by
Status: | review → pending |
---|
comment:10 Changed 13 years ago by
Owner: | Martin Kou deleted |
---|---|
Status: | pending → confirmed |
And I added a patch. Some part of the code in the patch is related to http://dev.fckeditor.net/attachment/ticket/2778/2778.patch