Tabs: Removed select method. Fixes #7156 - Tabs: Remove select method.

jquery · Oct 25, 2012 · 7cf2719 · 7cf2719
1 parent 0fc682a
commit 7cf2719
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 60 deletions.
diff --git a/tests/unit/tabs/tabs_deprecated.js b/tests/unit/tabs/tabs_deprecated.js
@@ -391,51 +391,6 @@ test( "remove", function() {
 	equal( element.tabs( "option", "active" ), false );
 });
 
-test( "select", function() {
-	expect( 23 );
-
-	var element = $( "#tabs1" ).tabs();
-	state( element, 1, 0, 0 );
-	element.tabs( "select", 1 );
-	state( element, 0, 1, 0 );
-	equal( element.tabs( "option", "active" ), 1, "active" );
-	equal( element.tabs( "option", "selected" ), 1, "selected" );
-	element.tabs( "destroy" );
-
-	element.tabs({ collapsible: true });
-	state( element, 1, 0, 0 );
-	element.tabs( "select", 0 );
-	state( element, 0, 0, 0 );
-	equal( element.tabs( "option", "active" ), false, "active" );
-	equal( element.tabs( "option", "selected" ), -1, "selected" );
-	element.tabs( "destroy" );
-
-	element.tabs({ collapsible: true });
-	element.tabs( "select", -1 );
-	state( element, 0, 0, 0 );
-	equal( element.tabs( "option", "active" ), false, "active" );
-	equal( element.tabs( "option", "selected" ), -1, "selected" );
-	element.tabs( "destroy" );
-
-	element.tabs();
-	state( element, 1, 0, 0 );
-	equal( element.tabs( "option", "active" ), 0, "active" );
-	equal( element.tabs( "option", "selected" ), 0, "selected" );
-	element.tabs( "select", 0 );
-	state( element, 1, 0, 0 );
-	equal( element.tabs( "option", "active" ), 0, "active" );
-	equal( element.tabs( "option", "selected" ), 0, "selected" );
-	element.tabs( "select", -1 );
-	state( element, 1, 0, 0 );
-	equal( element.tabs( "option", "active" ), 0, "active" );
-	equal( element.tabs( "option", "selected" ), 0, "selected" );
-
-	element.tabs( "select", "#fragment-2" );
-	state( element, 0, 1, 0 );
-	equal( element.tabs( "option", "active" ), 1, "active" );
-	equal( element.tabs( "option", "selected" ), 1, "selected" );
-});
-
 asyncTest( "abort", function() {
 	expect( 1 );
 

diff --git a/ui/jquery.ui.tabs.js b/ui/jquery.ui.tabs.js
@@ -1082,21 +1082,6 @@ if ( $.uiBackCompat !== false ) {
 		}
 	});
 
-	// select method
-	$.widget( "ui.tabs", $.ui.tabs, {
-		select: function( index ) {
-			index = this._getIndex( index );
-			if ( index === -1 ) {
-				if ( this.options.collapsible && this.options.selected !== -1 ) {
-					index = this.options.selected;
-				} else {
-					return;
-				}
-			}
-			this.anchors.eq( index ).trigger( this.options.event + this.eventNamespace );
-		}
-	});
-
 	// load event
 	$.widget( "ui.tabs", $.ui.tabs, {
 		_trigger: function( type, event, data ) {