Ticket #4480: 4480.patch

_source/plugins/table/dialogs/table.js

@@ -97 +97 @@
                                                 }
                                         }
 
-                                        // Modify the table headers. Depends on havint rows and cols generated
+                                        // Modify the table headers. Depends on having rows and cols generated
                                         // correctly so it can't be done in commit functions.
 
                                         // Should we make a <thead>?
@@ -115 +115 @@
                                                         if ( th.type == CKEDITOR.NODE_ELEMENT )
                                                         {
                                                                 th.renameNode( 'th' );
-                                                                if ( !i )
-                                                                        th.setAttribute( 'scope', 'col' );
+                                                                th.setAttribute( 'scope', 'col' );
                                                         }
                                                 }
                                                 thead.append( theRow.remove() );
@@ -153 +152 @@
                                                 {
                                                         newCell = new CKEDITOR.dom.element( table.$.rows[ row ].cells[ 0 ] );
                                                         newCell.renameNode( 'th' );
-                                                        newCell.setAttribute( 'scope', 'col' );
+                                                        newCell.setAttribute( 'scope', 'row' );
                                                 }
                                         }
 
@@ -166 +165 @@
                                                         if ( row.getParent().getName() == 'tbody' )
                                                         {
                                                                 newCell = new CKEDITOR.dom.element( row.$.cells[0] );
-                                                                newCell.renameNode( 'td');
+                                                                newCell.renameNode( 'td' );
                                                                 newCell.removeAttribute( 'scope' );
                                                         }
                                                 }

CHANGES.html

@@ -71 +71 @@
                 <li><a href="http://dev.fckeditor.net/ticket/4385">#4385</a> : Fixed editor's auto adjusting on DOM structure were confusing the dirty checking mechanism.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/4397">#4397</a> : Fixed regression of [3816] where turn on design mode is causing Firefox3 to scroll the host page.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/4254">#4254</a> : Added basic API sample.</li> 
+                <li><a href="http://dev.fckeditor.net/ticket/4480">#4480</a> : Fixed scope attribute in th.</li> 
         </ul>
         <h3>
                 CKEditor 3.0</h3>