github/jquery-ui
1
1
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2026-04-20 03:02:41 -04:00
Code Issues Packages Projects Releases Wiki Activity

Tabs: Use event.preventDefault() instead of return false. Fixes #6765 - Tabs: Use event.preventDefault() instead of returning false.

Browse Source
This commit is contained in:
MaxPower15
2010-12-20 08:50:23 -05:00
committed by Scott González
parent 956697bfca
commit e104f6cfa0
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
ui/jquery.ui.tabs.js vendored Normal file → Executable file
View File

@@ -330,7 +330,8 @@ $.widget( "ui.tabs", {
};
// attach tab event handler, unbind to avoid duplicates from former tabifying...
this.anchors.bind( o.event + ".tabs", function() {
this.anchors.bind( o.event + ".tabs", function( event ) {
event.preventDefault();
var el = this,
$li = $(el).closest( "li" ),
$hide = self.panels.filter( ":not(.ui-tabs-hide)" ),
@@ -346,7 +347,7 @@ $.widget( "ui.tabs", {
self.panels.filter( ":animated" ).length ||
self._trigger( "select", null, self._ui( this, $show[ 0 ] ) ) === false ) {
this.blur();
return false;
return;
}
o.selected = self.anchors.index( this );
@@ -367,7 +368,7 @@ $.widget( "ui.tabs", {
}).dequeue( "tabs" );
this.blur();
return false;
return;
} else if ( !$hide.length ) {
if ( o.cookie ) {
self._cookie( o.selected, o.cookie );
@@ -381,7 +382,7 @@ $.widget( "ui.tabs", {
self.load( self.anchors.index( this ) );
this.blur();
return false;
return;
}
}
@@ -415,8 +416,8 @@ $.widget( "ui.tabs", {
});
// disable click in any case
this.anchors.bind( "click.tabs", function(){
return false;
this.anchors.bind( "click.tabs", function( event ){
event.preventDefault();
});
},