Changeset 4987


Ignore:
Timestamp:
01/26/10 15:15:05 (4 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 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy