Opened 15 years ago

Closed 15 years ago

#2874 closed Bug (fixed)

tables: in FF caption seems to be in the wrong place

Reported by: trumbitta Owned by: Martin Kou
Priority: Normal Milestone: FCKeditor 2.6.5
Component: General Version: FCKeditor 2.6.4
Keywords: Confirmed HasPatch Review+ Cc:

Description

FCKeditor generates the following

    <table ....>
        <thead>
            <tr>
                <th scope="col">1</th>
                <th scope="col">2</th>
            </tr>
        </thead>
        <caption>My Caption</caption>
        <tbody>
            <tr>
                <td>...</td>
                <td>...</td>
            </tr>
        </tbody>
    </table>

The following would be the correct markup

    <table ....>
        <caption>My Caption</caption>
        <thead>
            <tr>
                <th scope="col">1</th>
                <th scope="col">2</th>
            </tr>
        </thead>
        <tbody>
            <tr>
                <td>...</td>
                <td>...</td>
            </tr>
        </tbody>
    </table>

Going further with my tests, I just found that the bug doesn't show up if you use MSIE7. My conclusions:

  • the problem is strictly related to the browser and its "DOM engine"

OR

  • the problem is inside the portion of code dedicated to gecko

Attachments (2)

2874.patch (2.5 KB) - added by Martin Kou 15 years ago.
2874_2.patch (837 bytes) - added by Koen Willems 15 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 15 years ago by Frederico Caldeira Knabben

Keywords: Confirmed HasPatch added

comment:2 Changed 15 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

Changed 15 years ago by Martin Kou

Attachment: 2874.patch added

comment:3 Changed 15 years ago by Martin Kou

Keywords: Review? added

comment:4 Changed 15 years ago by Garry Yao

Keywords: Review+ added; Review? removed

comment:5 Changed 15 years ago by Martin Kou

Resolution: fixed
Status: assignedclosed

Fixed with [3897].

Click here for more info about our SVN system.

comment:6 Changed 15 years ago by Koen Willems

Please re-open this one. Tested it in Firefox 3.5 but it still produces an invalid result.

To reproduce: Create a table with a caption but without any thead and th-cell. Then, edit the table and make the first row a thead filled with th-cells. In that case, the caption is placed after the thead.

Changed 15 years ago by Koen Willems

Attachment: 2874_2.patch added

comment:7 Changed 15 years ago by Koen Willems

Added a patch. As far as I can see all is working fine now!

comment:8 Changed 15 years ago by Koen Willems

Resolution: fixed
Status: closedreopened

comment:9 Changed 15 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: reopenedclosed

@kwillems, thanks for the fix. Committed with [4229].

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