Ticket #9101: 9101_proposal.patch.patch

File 9101_proposal.patch.patch, 2.0 KB (added by Damian, 7 years ago)

Propsed patch to address tab issue, along with role="presentation" on structural elements that should have it

  • plugins/smiley/dialogs/smiley.js

     
    8888
    8989                        // RIGHT-ARROW
    9090                        case rtl ? 37 : 39 :
    91                         // TAB
    92                         case 9 :
    9391                                // relative is TD
    9492                                if ( ( relative = element.getParent().getNext() ) )
    9593                                {
     
    109107
    110108                        // LEFT-ARROW
    111109                        case rtl ? 39 : 37 :
    112                         // SHIFT + TAB
    113                         case CKEDITOR.SHIFT + 9 :
    114110                                // relative is TD
    115111                                if ( ( relative = element.getParent().getPrevious() ) )
    116112                                {
     
    147143        for ( i = 0 ; i < size ; i++ )
    148144        {
    149145                if ( i % columns === 0 )
    150                         html.push( '<tr>' );
     146                        html.push( '<tr role="presentation">' );
    151147
    152148                var smileyLabelId = 'cke_smile_label_' + i + '_' + CKEDITOR.tools.getNextNumber();
    153149                html.push(
    154                         '<td class="cke_dark_background cke_centered" style="vertical-align: middle;">' +
     150                        '<td class="cke_dark_background cke_centered" style="vertical-align: middle;" role="presentation">' +
    155151                                '<a href="javascript:void(0)" role="option"',
    156152                                        ' aria-posinset="' + ( i +1 ) + '"',
    157153                                        ' aria-setsize="' + size + '"',
  • plugins/specialchar/dialogs/specialchar.js

     
    126126
    127127                        // RIGHT-ARROW
    128128                        case rtl ? 37 : 39 :
    129                         // TAB
    130                         case 9 :
    131129                                // relative is TD
    132130                                if ( ( relative = element.getParent().getNext() ) )
    133131                                {
     
    160158
    161159                        // LEFT-ARROW
    162160                        case rtl ? 39 : 37 :
    163                         // SHIFT + TAB
    164                         case CKEDITOR.SHIFT + 9 :
    165161                                // relative is TD
    166162                                if ( ( relative = element.getParent().getPrevious() ) )
    167163                                {
     
    213209
    214210                        while ( i < size )
    215211                        {
    216                                 html.push( '<tr>' ) ;
     212                                html.push( '<tr role="presentation">' ) ;
    217213
    218214                                for ( var j = 0 ; j < columns ; j++, i++ )
    219215                                {
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy