Changeset 4987


Ignore:
Timestamp:
01/26/10 15:15:05 (6 years ago)
Author:
garry.yao
Message:

Fixing lint errors.

Location:
CKEditor/branches/features/aria/_source/plugins
Files:
7 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/branches/features/aria/_source/plugins/accessibility/plugin.js

    r4978 r4987  
    4040                // the role is already supported on it.
    4141                if( lookupARIASupport( role, element.getName() ) !== false )
    42                         return element;
     42                        return;
    4343
    4444                var attrValue,
     
    7171                        fieldset.insertBefore( element );
    7272                        fieldset.append( element );
    73                         return fieldset;
    7473                }
    7574                // The only reliable substitution of aria-label on an iframe
     
    7776                else if ( element.is( 'iframe' ) )
    7877                {
    79                         var doc = element.$.contentWindow.document,
    80                                 title = doc.title;
     78                        doc = element.$.contentWindow.document;
     79                        var title = doc.title;
    8180
    8281                        // Backup the title and restore it before running into use.
     
    112111                                                        labelElement.setHtml( htmlTitle );
    113112                                                }
    114                                         }
     113                                        };
    115114                                } );
    116115                        }
     
    123122                                        var target = evt.data,
    124123                                                ariaLabel;
    125                                         if( ariaLabel = target.getAttribute( 'aria-label' ) )
     124                                        if( ( ariaLabel = target.getAttribute( 'aria-label' ) ) )
    126125                                        {
    127126                                                var labelId = 'cke_label_' + CKEDITOR.tools.getNextNumber();
  • CKEditor/branches/features/aria/_source/plugins/button/plugin.js

    r4959 r4987  
    132132                        classes += ' ' + this.className;
    133133
    134                 var env = CKEDITOR.env;
     134                env = CKEDITOR.env;
    135135                output.push(
    136136                        '<span class="cke_button">',
     
    197197        {
    198198                if ( this._.state == state )
    199                         return;
     199                        return false;
     200
     201                this._.state = state;
    200202
    201203                var element = CKEDITOR.document.getById( this._.id );
     
    214216                        return true;
    215217                }
    216 
    217                 this._.state = state;
     218                else
     219                        return false;
    218220        }
    219221};
  • CKEditor/branches/features/aria/_source/plugins/clipboard/dialogs/paste.js

    r4957 r4987  
    109109                        container.append( iframe );
    110110
    111                         field.getInputElement = function(){ return iframe };
     111                        field.getInputElement = function(){ return iframe; };
    112112
    113113                        // Force container to scale in IE.
  • CKEditor/branches/features/aria/_source/plugins/floatpanel/plugin.js

    r4977 r4987  
    182182                                panel.onEscape = CKEDITOR.tools.bind( function( keystroke )
    183183                                        {
    184                                                 if ( this.onEscape && this.onEscape( keystroke ) === false );
     184                                                if ( this.onEscape && this.onEscape( keystroke ) === false )
    185185                                                        return false;
    186186                                        },
  • CKEditor/branches/features/aria/_source/plugins/image/dialogs/image.js

    r4954 r4987  
    150150                var lang = dialog._.editor.lang.image,
    151151                        label =  lang[  dialog.lockRatio ? 'unlockRatio' : 'lockRatio' ];
    152                 ratioButton.setAttribute( 'title', label )
     152
     153                ratioButton.setAttribute( 'title', label );
    153154                ratioButton.getFirst().setText( label );
    154155
  • CKEditor/branches/features/aria/_source/plugins/panel/plugin.js

    r4977 r4987  
    183183                                                // ESC/ARROW-LEFT(ltr) OR ARROW-RIGHT(rtl)
    184184                                                if ( keystroke == 27 || keystroke == ( dir == 'rtl' ? 39 : 37 ) )
    185                                                         if ( this.onEscape && this.onEscape( keystroke ) === false );
    186                                                                 evt.data.preventDefault();
     185                                                {
     186                                                        if ( this.onEscape && this.onEscape( keystroke ) === false )
     187                                                                evt.data.preventDefault( );
     188                                                }
    187189                                        },
    188190                                        this );
  • CKEditor/branches/features/aria/_source/plugins/templates/dialogs/templates.js

    r4963 r4987  
    130130                                        }
    131131                                }
    132                         };
     132                        }
    133133
    134134                        // Load skin at first.
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy