Opened 11 years ago

Last modified 8 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)

2859.patch (3.3 KB) - added by Koen Willems 11 years ago.
2859_2.patch (7.3 KB) - added by Martin Kou 10 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 11 years ago by Koen Willems

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

Changed 11 years ago by Koen Willems

Attachment: 2859.patch added

comment:2 Changed 11 years ago by Artur Formella

Component: GeneralCore : Output Data
Keywords: Confirmed added
Version: FCKeditor 2.6.4FCKeditor 2.3.2

comment:3 Changed 11 years ago by Koen Willems

Cc: kwillems@… added

comment:4 Changed 10 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

Changed 10 years ago by Martin Kou

Attachment: 2859_2.patch added

comment:5 Changed 10 years ago by Martin Kou

Keywords: Review? added

The proposed patch includes the changes in #2813.

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

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 9 years ago by Wiktor Walc

Component: Core : Output DataCore : Tables

comment:8 Changed 8 years ago by Jakub Ś

Keywords: Confirmed removed
Version: FCKeditor 2.3.23.0

This issue is also reproducible in latest CKEditor 3.6.2

comment:9 Changed 8 years ago by Jakub Ś

Status: reviewpending

comment:10 Changed 8 years ago by Jakub Ś

Owner: Martin Kou deleted
Status: pendingconfirmed
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy