diff options
author | Mike Sherov <mike.sherov@gmail.com> | 2012-10-23 09:36:42 -0500 |
---|---|---|
committer | Corey Frang <gnarf@gnarf.net> | 2012-10-23 09:37:46 -0500 |
commit | e1230997aa14dae6f35326c8ba20bfe2428507c2 (patch) | |
tree | f00984f5fbbf12f175c0994c497513a2d8e8ee26 /tests/unit/menu/menu_events.js | |
parent | 7af1ec727bcca8367e804cea77b9dd238b1c0d69 (diff) | |
download | jquery-ui-e1230997aa14dae6f35326c8ba20bfe2428507c2.tar.gz jquery-ui-e1230997aa14dae6f35326c8ba20bfe2428507c2.zip |
Build: Enable "unused" option in jshint - Remove unused variables from codebase. - Closes gh-788
Squashed commit of the following:
commit 7f19f92c646f180bc067bb24123175251a64a9d6
Author: Mike Sherov <mike.sherov@gmail.com>
Date: Tue Oct 23 10:34:28 2012 -0400
put back in fake args for signatures that we want to keep
commit 257505a9e69da0c53e3a989dab87a13112045a29
Author: Mike Sherov <mike.sherov@gmail.com>
Date: Tue Oct 23 08:10:20 2012 -0400
changes per @scott_gonzalez
commit 12725480cb58e70865e5aa6e735009b6b035c8f3
Author: Mike Sherov <mike.sherov@gmail.com>
Date: Mon Oct 22 22:54:05 2012 -0400
clean up unused vars in ui directory
commit 563595e7aee5d4a5c096b2d1de655abdf920aacd
Author: Mike Sherov <mike.sherov@gmail.com>
Date: Mon Oct 22 22:37:42 2012 -0400
clean up unused vars in grunt and tests
Diffstat (limited to 'tests/unit/menu/menu_events.js')
-rw-r--r-- | tests/unit/menu/menu_events.js | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/tests/unit/menu/menu_events.js b/tests/unit/menu/menu_events.js index d69fc46a3..a94968138 100644 --- a/tests/unit/menu/menu_events.js +++ b/tests/unit/menu/menu_events.js @@ -13,7 +13,7 @@ module( "menu: events", { test( "handle click on menu", function() { expect( 1 ); var element = $( "#menu1" ).menu({ - select: function( event, ui ) { + select: function() { log(); } }); @@ -29,7 +29,7 @@ test( "handle click on menu", function() { test( "handle click on custom item menu", function() { expect( 1 ); var element = $( "#menu5" ).menu({ - select: function( event, ui ) { + select: function() { log(); }, menus: "div" @@ -47,7 +47,7 @@ asyncTest( "handle blur", function() { expect( 1 ); var blurHandled = false, element = $( "#menu1" ).menu({ - blur: function( event, ui ) { + blur: function( event ) { // Ignore duplicate blur event fired by IE if ( !blurHandled ) { blurHandled = true; @@ -69,7 +69,7 @@ asyncTest( "handle blur via click outside", function() { expect( 1 ); var blurHandled = false, element = $( "#menu1" ).menu({ - blur: function( event, ui ) { + blur: function( event ) { // Ignore duplicate blur event fired by IE if ( !blurHandled ) { blurHandled = true; @@ -90,7 +90,7 @@ asyncTest( "handle blur via click outside", function() { test( "handle focus of menu with active item", function() { expect( 1 ); var element = $( "#menu1" ).menu({ - focus: function( event, ui ) { + focus: function( event ) { log( $( event.target ).find( ".ui-state-focus" ).parent().index() ); } }); @@ -168,7 +168,7 @@ test( "handle keyboard navigation on menu without scroll and without submenus", select: function( event, ui ) { log( $( ui.item[0] ).text() ); }, - focus: function( event, ui ) { + focus: function( event ) { log( $( event.target ).find( ".ui-state-focus" ).parent().index() ); } }); @@ -230,13 +230,13 @@ asyncTest( "handle keyboard navigation on menu without scroll and with submenus" select: function( event, ui ) { log( $( ui.item[0] ).text() ); }, - focus: function( event, ui ) { + focus: function( event ) { log( $( event.target ).find( ".ui-state-focus" ).parent().index() ); } }); log( "keydown", true ); - element.one( "menufocus", function( event, ui ) { + element.one( "menufocus", function() { element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } ); element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } ); equal( logOutput(), "keydown,1,2", "Keydown DOWN" ); @@ -350,7 +350,7 @@ test( "handle keyboard navigation on menu with scroll and without submenus", fun select: function( event, ui ) { log( $( ui.item[0] ).text() ); }, - focus: function( event, ui ) { + focus: function( event ) { log( $( event.target ).find( ".ui-state-focus" ).parent().index()); } }); @@ -421,13 +421,13 @@ asyncTest( "handle keyboard navigation on menu with scroll and with submenus", f select: function( event, ui ) { log( $( ui.item[0] ).text() ); }, - focus: function( event, ui ) { + focus: function( event ) { log( $( event.target ).find( ".ui-state-focus" ).parent().index()); } }); log( "keydown", true ); - element.one( "menufocus", function( event, ui ) { + element.one( "menufocus", function() { element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } ); element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } ); equal( logOutput(), "keydown,1,2", "Keydown DOWN" ); @@ -516,13 +516,13 @@ asyncTest( "handle keyboard navigation and mouse click on menu with disabled ite select: function( event, ui ) { log( $( ui.item[0] ).text() ); }, - focus: function( event, ui ) { + focus: function( event ) { log( $( event.target ).find( ".ui-state-focus" ).parent().index()); } }); log( "keydown", true ); - element.one( "menufocus", function( event, ui ) { + element.one( "menufocus", function() { element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } ); element.simulate( "keydown", { keyCode: $.ui.keyCode.ENTER } ); equal( logOutput(), "keydown,1", "Keydown focus but not select disabled item" ); @@ -564,13 +564,13 @@ asyncTest( "handle keyboard navigation and mouse click on menu with disabled ite test( "handle keyboard navigation with spelling of menu items", function() { expect( 2 ); var element = $( "#menu2" ).menu({ - focus: function( event, ui ) { + focus: function( event ) { log( $( event.target ).find( ".ui-state-focus" ).parent().index() ); } }); log( "keydown", true ); - element.one( "menufocus", function( event, ui ) { + element.one( "menufocus", function() { element.simulate( "keydown", { keyCode: 65 } ); element.simulate( "keydown", { keyCode: 68 } ); element.simulate( "keydown", { keyCode: 68 } ); |