Opened 17 years ago
Closed 14 years ago
#2813 closed Bug (invalid)
Deleting a row does not correct the rowspan of merged cells
| Reported by: | Koen Willems | Owned by: | Martin Kou |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | Core : Tables | Version: | FCKeditor 2.6.4 |
| Keywords: | HasPatch? | Cc: | kwillems@… |
Description
Take a table like this:
<table title="tabel" 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>
</tbody>
</table>
Now, put the cursor in cell a, right click and remove that row. We see that cell b disappears. Or go to cell b and remove that row: cell b is deleted and cells c and d are left over.
Attachments (5)
Change History (16)
Changed 17 years ago by
| Attachment: | 2813.patch added |
|---|
comment:1 Changed 17 years ago by
Changed 17 years ago by
| Attachment: | 2813_2.patch added |
|---|
comment:2 Changed 17 years ago by
Made a little improvement; there was an unneccesary check in the previous patch.
comment:3 Changed 17 years ago by
| Cc: | kwillems@… added |
|---|---|
| Milestone: | → FCKeditor 2.6.5 |
| Version: | → FCKeditor 2.6.4 |
comment:4 Changed 16 years ago by
| Keywords: | HasPatch added |
|---|---|
| Owner: | set to Martin Kou |
| Status: | new → assigned |
Changed 16 years ago by
| Attachment: | 2813_3.patch added |
|---|
comment:5 Changed 16 years ago by
| Keywords: | Review? added |
|---|
comment:6 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|
The patch is breaking the merge command when 2x2 cells are selected in the default table.
comment:7 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Changed 16 years ago by
| Attachment: | 2813_4.patch added |
|---|
Changed 16 years ago by
| Attachment: | 2813_5.patch added |
|---|
comment:8 Changed 16 years ago by
Found a typo in 2813_4.patch while working on #2859 - posted new patch which fixes it.
comment:9 Changed 16 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:10 Changed 15 years ago by
| Component: | General → Core : Tables |
|---|
comment:11 Changed 14 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | review → closed |
FCKeditor was retired and is no longer supported. All active development was moved to its successor, CKEditor 3.x, that is a fully mature and far superior product. We recommend you upgrade as soon as possible since this issue is no longer a problem in current code base.

And I added a patch.