]> source.dussan.org Git - jquery-ui.git/commitdiff
Menubar: Whitespace.
authorScott González <scott.gonzalez@gmail.com>
Sat, 28 May 2011 19:42:28 +0000 (15:42 -0400)
committerScott González <scott.gonzalez@gmail.com>
Sat, 28 May 2011 19:42:28 +0000 (15:42 -0400)
ui/jquery.ui.menubar.js

index 6716805977af87e37b733494b8f010f727f3302b..a0e9afb3c2b67d2f10efa31b613b5a8022dd0c92 100644 (file)
 // there has to be just one item that has tabindex
 $.widget( "ui.menubar", {
        version: "@VERSION",
-   options: {
-      buttons: false,
-      menuIcon: false
-   },
+       options: {
+               buttons: false,
+               menuIcon: false
+       },
        _create: function() {
                var that = this;
                var items = this.items = this.element.children( "li" )
@@ -69,13 +69,13 @@ $.widget( "ui.menubar", {
                        var input = $(this),
                                // TODO menu var is only used on two places, doesn't quite justify the .each
                                menu = input.next( "ul" );
-                       
+
                        input.bind( "click.menubar focus.menubar mouseenter.menubar", function( event ) {
                                // ignore triggered focus event
                                if ( event.type == "focus" && !event.originalEvent ) {
                                        return;
                                }
-                               event.preventDefault();
+                               event.preventDefault();
                                // TODO can we simplify or extractthis check? especially the last two expressions
                                // there's a similar active[0] == menu[0] check in _open
                                if ( event.type == "click" && menu.is( ":visible" ) && that.active && that.active[0] == menu[0] ) {
@@ -85,7 +85,7 @@ $.widget( "ui.menubar", {
                                if ( ( that.open && event.type == "mouseenter" ) || event.type == "click" ) {
                                        that._open( event, menu );
                                }
-                       })
+                       })
                        .bind( "keydown", function( event ) {
                                switch ( event.keyCode ) {
                                case $.ui.keyCode.SPACE:
@@ -114,11 +114,11 @@ $.widget( "ui.menubar", {
                                input.addClass( "ui-state-default" ).append( "<span class='ui-button-icon-secondary ui-icon ui-icon-triangle-1-s'></span>" );
                                input.removeClass( "ui-button-text-only" ).addClass( "ui-button-text-icon-secondary" );
                        }
-                       
+
                        if ( !that.options.buttons ) {
                                // TODO ui-menubar-link is added above, not needed here?
                                input.addClass( "ui-menubar-link" ).removeClass( "ui-state-default" );
-                       };                      
+                       };
                        
                });
                that._bind( {
@@ -140,18 +140,18 @@ $.widget( "ui.menubar", {
                        }
                });
        },
-       
+
        _destroy : function() {
                var items = this.element.children( "li" )
                        .removeClass( "ui-menubar-item" )
                        .removeAttr( "role", "presentation" )
                        .children( "button, a" );
-               
+
                this.element
                        .removeClass( "ui-menubar ui-widget-header ui-helper-clearfix" )
                        .removeAttr( "role", "menubar" )
                        .unbind( ".menubar" );
-               
+
                items
                        .unbind( ".menubar" )
                        .removeClass( "ui-button ui-widget ui-button-text-only ui-menubar-link ui-state-default" )
@@ -173,7 +173,7 @@ $.widget( "ui.menubar", {
                        .removeAttr( "tabindex" )
                        .unbind( ".menubar" );
        },
-       
+
        _close: function() {
                if ( !this.active || !this.active.length )
                        return;
@@ -189,7 +189,7 @@ $.widget( "ui.menubar", {
                this.active = null;
                this.open = false;
        },
-       
+
        _open: function( event, menu ) {
                // on a single-button menubar, ignore reopening the same menu
                if ( this.active && this.active[0] == menu[0] ) {
@@ -223,7 +223,7 @@ $.widget( "ui.menubar", {
                        .focusin();
                this.open = true;
        },
-       
+
        // TODO refactor this and the next three methods
        _prev: function( event, button ) {
                button.attr( "tabIndex", -1 );
@@ -235,7 +235,7 @@ $.widget( "ui.menubar", {
                        lastItem.removeAttr( "tabIndex" )[0].focus();
                }
        },
-       
+
        _next: function( event, button ) {
                button.attr( "tabIndex", -1 );
                var next = button.parent().nextAll( "li" ).children( ".ui-button" ).eq( 0 );
@@ -257,7 +257,7 @@ $.widget( "ui.menubar", {
                        this._open( event, lastItem );
                }
        },
-       
+
        // TODO rename to child (or something like that)
        _right: function( event ) {
                var next = this.active.parent().nextAll( "li:eq(0)" ).children( ".ui-menu" ).eq( 0 );