From 62e00921564ff979b00c0695a8622be04c2ac686 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jo=CC=88rn=20Zaefferer?= Date: Sat, 14 Mar 2015 12:28:17 +0100 Subject: [PATCH] Selectmenu: Fix style issues Closes gh-1498 --- ui/selectmenu.js | 128 +++++++++++++++++++++++------------------------ 1 file changed, 64 insertions(+), 64 deletions(-) diff --git a/ui/selectmenu.js b/ui/selectmenu.js index 0d1354c6d..74e142d33 100644 --- a/ui/selectmenu.js +++ b/ui/selectmenu.js @@ -16,11 +16,11 @@ //>>css.structure: ../themes/base/selectmenu.css //>>css.theme: ../themes/base/theme.css -(function( factory ) { +( function( factory ) { if ( typeof define === "function" && define.amd ) { // AMD. Register as an anonymous module. - define([ + define( [ "jquery", "./core", "./widget", @@ -32,7 +32,7 @@ // Browser globals factory( jQuery ); } -}(function( $ ) { +}( function( $ ) { return $.widget( "ui.selectmenu", { version: "@VERSION", @@ -96,7 +96,7 @@ return $.widget( "ui.selectmenu", { this.button.focus(); event.preventDefault(); } - }); + } ); // Hide original select element this.element.hide(); @@ -111,7 +111,7 @@ return $.widget( "ui.selectmenu", { "aria-owns": this.ids.menu, "aria-haspopup": "true", title: this.element.attr( "title" ) - }) + } ) .insertAfter( this.element ); this._addClass( this.button, "ui-selectmenu-button ui-selectmenu-button-closed", @@ -135,7 +135,7 @@ return $.widget( "ui.selectmenu", { if ( !that._rendered ) { that._refreshMenu(); } - }); + } ); this._hoverable( this.button ); this._focusable( this.button ); }, @@ -148,7 +148,7 @@ return $.widget( "ui.selectmenu", { "aria-hidden": "true", "aria-labelledby": this.ids.button, id: this.ids.menu - }); + } ); // Wrap menu this.menuWrap = $( "
" ).append( this.menu ); @@ -157,7 +157,7 @@ return $.widget( "ui.selectmenu", { // Initialize menu widget this.menuInstance = this.menu - .menu({ + .menu( { classes: { "ui-menu": "ui-corner-bottom" }, @@ -187,7 +187,7 @@ return $.widget( "ui.selectmenu", { that.button.attr( "aria-activedescendant", that.menuItems.eq( item.index ).attr( "id" ) ); } - }) + } ) .menu( "instance" ); // Don't close the menu on mouseleave @@ -323,7 +323,7 @@ return $.widget( "ui.selectmenu", { if ( item.optgroup !== currentOptgroup ) { li = $( "
  • ", { text: item.optgroup - }); + } ); that._addClass( li, "ui-selectmenu-optgroup", "ui-menu-divider" + ( item.element.parent( "optgroup" ).prop( "disabled" ) ? " ui-state-disabled" : @@ -335,7 +335,7 @@ return $.widget( "ui.selectmenu", { } that._renderItemData( ul, item ); - }); + } ); }, _renderItemData: function( ul, item ) { @@ -346,7 +346,7 @@ return $.widget( "ui.selectmenu", { var li = $( "
  • " ), wrapper = $( "
    ", { title: item.element.attr( "title" ) - }); + } ); if ( item.disabled ) { this._addClass( li, null, "ui-state-disabled" ); @@ -455,54 +455,54 @@ return $.widget( "ui.selectmenu", { keydown: function( event ) { var preventDefault = true; switch ( event.keyCode ) { - case $.ui.keyCode.TAB: - case $.ui.keyCode.ESCAPE: - this.close( event ); - preventDefault = false; - break; - case $.ui.keyCode.ENTER: - if ( this.isOpen ) { - this._selectFocusedItem( event ); - } - break; - case $.ui.keyCode.UP: - if ( event.altKey ) { - this._toggle( event ); - } else { - this._move( "prev", event ); - } - break; - case $.ui.keyCode.DOWN: - if ( event.altKey ) { - this._toggle( event ); - } else { - this._move( "next", event ); - } - break; - case $.ui.keyCode.SPACE: - if ( this.isOpen ) { - this._selectFocusedItem( event ); - } else { - this._toggle( event ); - } - break; - case $.ui.keyCode.LEFT: + case $.ui.keyCode.TAB: + case $.ui.keyCode.ESCAPE: + this.close( event ); + preventDefault = false; + break; + case $.ui.keyCode.ENTER: + if ( this.isOpen ) { + this._selectFocusedItem( event ); + } + break; + case $.ui.keyCode.UP: + if ( event.altKey ) { + this._toggle( event ); + } else { this._move( "prev", event ); - break; - case $.ui.keyCode.RIGHT: + } + break; + case $.ui.keyCode.DOWN: + if ( event.altKey ) { + this._toggle( event ); + } else { this._move( "next", event ); - break; - case $.ui.keyCode.HOME: - case $.ui.keyCode.PAGE_UP: - this._move( "first", event ); - break; - case $.ui.keyCode.END: - case $.ui.keyCode.PAGE_DOWN: - this._move( "last", event ); - break; - default: - this.menu.trigger( event ); - preventDefault = false; + } + break; + case $.ui.keyCode.SPACE: + if ( this.isOpen ) { + this._selectFocusedItem( event ); + } else { + this._toggle( event ); + } + break; + case $.ui.keyCode.LEFT: + this._move( "prev", event ); + break; + case $.ui.keyCode.RIGHT: + this._move( "next", event ); + break; + case $.ui.keyCode.HOME: + case $.ui.keyCode.PAGE_UP: + this._move( "first", event ); + break; + case $.ui.keyCode.END: + case $.ui.keyCode.PAGE_DOWN: + this._move( "last", event ); + break; + default: + this.menu.trigger( event ); + preventDefault = false; } if ( preventDefault ) { @@ -537,10 +537,10 @@ return $.widget( "ui.selectmenu", { _setAria: function( item ) { var id = this.menuItems.eq( item.index ).attr( "id" ); - this.button.attr({ + this.button.attr( { "aria-labelledby": id, "aria-activedescendant": id - }); + } ); this.menu.attr( "aria-activedescendant", id ); }, @@ -647,9 +647,9 @@ return $.widget( "ui.selectmenu", { _parseOptions: function( options ) { var that = this, data = []; - options.each(function( index, item ) { + options.each( function( index, item ) { data.push( that._parseOption( $( item ), index ) ); - }); + } ); this.items = data; }, @@ -673,6 +673,6 @@ return $.widget( "ui.selectmenu", { this.element.removeUniqueId(); this.label.attr( "for", this.ids.element ); } -}); +} ); -})); +} ) ); -- 2.39.5