github/jquery-ui
1
1
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2026-01-29 11:18:05 -05:00
Code Issues Packages Projects Releases Wiki Activity

UI Tabs: reviewed code to handle hover/focus states: needs to take disabling into account, namespaced events, to be destroyed

Browse Source
This commit is contained in:
Klaus Hartl
2009-01-10 10:47:28 +00:00
parent 842753690a
commit 422e0666e9
1 changed files with 14 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

35
ui/ui.tabs.js
View File

@@ -32,7 +32,7 @@ $.widget("ui.tabs", {
$this.removeData(prefix + '.tabs');
});
});
this.$lis.add(this.$panels).each(function() {
this.$lis.unbind('.tabs').add(this.$panels).each(function() {
if ($.data(this, 'destroy.tabs'))
$(this).remove();
else
@@ -78,24 +78,6 @@ $.widget("ui.tabs", {
var self = this, o = this.options;
this.$lis
.hover(
function() {
$(this).addClass('ui-state-hover');
},
function() {
$(this).removeClass('ui-state-hover');
}
);
this.$tabs
.focus(function() {
$(this).parent().addClass('ui-state-focus');
})
.blur(function() {
$(this).parent().removeClass('ui-state-focus');
});
this.$tabs.each(function(i, a) {
// inline tab
if (a.hash && a.hash.replace('#', '')) // Safari 2 reports '#' for an empty hash
@@ -188,10 +170,21 @@ $.widget("ui.tabs", {
// just trigger show event
else onShow();
}
// states
var handleState = function(state, el) {
if (el.is(':not(.' + o.disabledClass + ')')) el.toggleClass('ui-state-' + state);
};
this.$lis.bind('mouseover.tabs mouseout.tabs', function() {
handleState('hover', $(this));
});
this.$tabs.bind('focus.tabs blur.tabs', function() {
handleState('focus', $(this).parents('li:first'));
});
// clean up to avoid memory leaks in certain versions of IE 6
$(window).bind('unload', function() {
self.$tabs.unbind('.tabs');
self.$lis.add(self.$tabs).unbind('.tabs');
self.$lis = self.$tabs = self.$panels = null;
});
@@ -342,7 +335,7 @@ $.widget("ui.tabs", {
return false;
});
// disable click if event is configured to something else
if (o.event != 'click') this.$tabs.bind('click.tabs', function(){return false;});