Opened 15 years ago

Last modified 12 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 15 years ago.
2859_2.patch (7.3 KB) - added by Martin Kou 15 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 15 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 15 years ago by Koen Willems

Attachment: 2859.patch added

comment:2 Changed 15 years ago by Artur Formella

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

comment:3 Changed 15 years ago by Koen Willems

Cc: kwillems@… added

comment:4 Changed 15 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

Changed 15 years ago by Martin Kou

Attachment: 2859_2.patch added

comment:5 Changed 15 years ago by Martin Kou

Keywords: Review? added

The proposed patch includes the changes in #2813.

comment:6 Changed 15 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 13 years ago by Wiktor Walc

Component: Core : Output DataCore : Tables

comment:8 Changed 12 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 12 years ago by Jakub Ś

Status: reviewpending

comment:10 Changed 12 years ago by Jakub Ś

Owner: Martin Kou deleted
Status: pendingconfirmed
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy