Ticket #4480: 4480.patch

File 4480.patch, 2.1 KB (added by Alfonso Martínez de Lizarrondo, 10 years ago)

Proposed patch

  • _source/plugins/table/dialogs/table.js

     
    9797                                                }
    9898                                        }
    9999
    100                                         // Modify the table headers. Depends on havint rows and cols generated
     100                                        // Modify the table headers. Depends on having rows and cols generated
    101101                                        // correctly so it can't be done in commit functions.
    102102
    103103                                        // Should we make a <thead>?
     
    115115                                                        if ( th.type == CKEDITOR.NODE_ELEMENT )
    116116                                                        {
    117117                                                                th.renameNode( 'th' );
    118                                                                 if ( !i )
    119                                                                         th.setAttribute( 'scope', 'col' );
     118                                                                th.setAttribute( 'scope', 'col' );
    120119                                                        }
    121120                                                }
    122121                                                thead.append( theRow.remove() );
     
    153152                                                {
    154153                                                        newCell = new CKEDITOR.dom.element( table.$.rows[ row ].cells[ 0 ] );
    155154                                                        newCell.renameNode( 'th' );
    156                                                         newCell.setAttribute( 'scope', 'col' );
     155                                                        newCell.setAttribute( 'scope', 'row' );
    157156                                                }
    158157                                        }
    159158
     
    166165                                                        if ( row.getParent().getName() == 'tbody' )
    167166                                                        {
    168167                                                                newCell = new CKEDITOR.dom.element( row.$.cells[0] );
    169                                                                 newCell.renameNode( 'td');
     168                                                                newCell.renameNode( 'td' );
    170169                                                                newCell.removeAttribute( 'scope' );
    171170                                                        }
    172171                                                }
  • CHANGES.html

     
    7171                <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>
    7272                <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>
    7373                <li><a href="http://dev.fckeditor.net/ticket/4254">#4254</a> : Added basic API sample.</li>
     74                <li><a href="http://dev.fckeditor.net/ticket/4480">#4480</a> : Fixed scope attribute in th.</li>
    7475        </ul>
    7576        <h3>
    7677                CKEditor 3.0</h3>
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy