Changeset 2283

Timestamp:

2008-07-25 07:47:28 (6 months ago)

Author:

martinkou

Message:

Fixed #2394: Fixed JavaScript error with the "split vertically" command in IE when attempting to split cells in the last row of a table.

Location:

FCKeditor/trunk

Files:

• editor/_source/internals/fcktablehandler.js (modified) (1 diff)
• _whatsnew.html (modified) (1 diff)

FCKeditor/trunk/editor/_source/internals/fcktablehandler.js

@@ -551 +551 @@
                 var newCellRowIndex = currentRowIndex + 1 ;
                 var newRow = FCK.EditorDocument.createElement( 'tr' ) ;
-                currentCell.parentNode.parentNode.insertBefore( newRow, currentCell.parentNode.parentNode.rows[newCellRowIndex] ) ;
+                var tBody = currentCell.parentNode.parentNode ;
+                if ( tBody.rows.length > newCellRowIndex )
+                        tBody.insertBefore( newRow, tBody.rows[newCellRowIndex] ) ;
+                else
+                        tBody.appendChild( newRow ) ;
 
                 // 2. +1 to rowSpan for all cells crossing currentCell's row.

FCKeditor/trunk/_whatsnew.html

@@ -110 +110 @@
                         source logic for comments in IE.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2387">#2387</a>] Fixed JavaScript
-                        with list commands when the editable document is selected with Ctrl-A.</li>
+                        error with list commands when the editable document is selected with Ctrl-A.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2390">#2390</a>] Fixed the issue where 
                         indent styles in JavaScript-generated &lt;p&gt; blocks are erased in IE.</li> 
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2394">#2394</a>] Fixed JavaScript
+                        error with the "split vertically" command in IE when attempting to split cells in the last
+                        row of a table.</li>
         </ul>
         <p>