Opened 17 years ago

Closed 13 years ago

#2487 closed Bug (invalid)

Split non-header cell in table with <thead> throws JavaScript error

Reported by: shri Owned by:
Priority: Normal Milestone:
Component: General Version: FCKeditor 2.4
Keywords: Cc:

Description

Tested below steps on trunk samples default page in FF and IE

  1. Create a 3x2 table with first row header.
  2. Split non-header cell horizontally and JS error is thrown.
  3. Also splitting non-header cell vertically breaks the table layout.
Find this issue on GitHub

Change History (4)

comment:1 Changed 17 years ago by Artur Formella

Keywords: Confirmed added
Version: SVNFCKeditor 2.4
  1. Insert source:
    <table cellspacing="1" cellpadding="1" border="1" width="200">
        <thead>
            <tr>
                <td>11</td>
                <td>22</td>
            </tr>
        </thead>
        <tbody>
            <tr>
                <td>33&nbsp;</td>
                <td>44&nbsp;</td>
            </tr>
            <tr>
                <td>55&nbsp;</td>
                <td>66&nbsp;</td>
            </tr>
        </tbody>
    </table>
    <p>&nbsp;</p>
    
  2. In WYSIWYG. right-click 33 > Cell > Split cell horizontally

You've got an error:

cell is undefined
http://localhost/edytor/FCKeditor/tags/2.6.3/editor/_source/internals/fcktablehandler.js
Line 702
701 var cell = tableMap[i][j] ;
702 if ( cell.parentNode )
703 cell.parentNode.removeChild( cell ) ;
704 cell.colSpan = cell[rowSpanAttr] = 1 ;

comment:2 Changed 17 years ago by Frederico Caldeira Knabben

May be related to #1865

comment:3 Changed 16 years ago by Koen Willems

comment:4 Changed 13 years ago by Jakub Ś

Resolution: invalid
Status: confirmedclosed

Does not occur in CKEditor.

Since FCKeditor is no longer supported and recommend upgrading to CKEditor I'm closing this ticket as it is no longer valid.

Find this issue on GitHub
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