Skip to content
Permalink
Browse files

Tabs: Never remove the ui-state-default class when switching tabs.

Fixes #4815 - Tabs: each tab needs to have state ui-state-default.
  • Loading branch information
scottgonzalez committed Nov 13, 2009
1 parent ef02b16 commit 18e256849cad7b0d439fd044b5693bbabe3cd946
Showing with 4 additions and 4 deletions.
  1. +4 −4 ui/jquery.ui.tabs.js
@@ -273,15 +273,15 @@ $.widget("ui.tabs", {
// Show a tab...
var showTab = showFx ?
function(clicked, $show) {
$(clicked).closest('li').removeClass('ui-state-default').addClass('ui-tabs-selected ui-state-active');
$(clicked).closest('li').addClass('ui-tabs-selected ui-state-active');
$show.hide().removeClass('ui-tabs-hide') // avoid flicker that way
.animate(showFx, showFx.duration || 'normal', function() {
resetStyle($show, showFx);
self._trigger('show', null, self._ui(clicked, $show[0]));
});
} :
function(clicked, $show) {
$(clicked).closest('li').removeClass('ui-state-default').addClass('ui-tabs-selected ui-state-active');
$(clicked).closest('li').addClass('ui-tabs-selected ui-state-active');
$show.removeClass('ui-tabs-hide');
self._trigger('show', null, self._ui(clicked, $show[0]));
};
@@ -290,14 +290,14 @@ $.widget("ui.tabs", {
var hideTab = hideFx ?
function(clicked, $hide) {
$hide.animate(hideFx, hideFx.duration || 'normal', function() {
self.lis.removeClass('ui-tabs-selected ui-state-active').addClass('ui-state-default');
self.lis.removeClass('ui-tabs-selected ui-state-active');
$hide.addClass('ui-tabs-hide');
resetStyle($hide, hideFx);
self.element.dequeue("tabs");
});
} :
function(clicked, $hide, $show) {
self.lis.removeClass('ui-tabs-selected ui-state-active').addClass('ui-state-default');
self.lis.removeClass('ui-tabs-selected ui-state-active');
$hide.addClass('ui-tabs-hide');
self.element.dequeue("tabs");
};

0 comments on commit 18e2568

Please sign in to comment.
You can’t perform that action at this time.