Add active class to the currently active tab content, as well as the tab itself

If this is not done, then the active class remains on the tab that was initially visible on page load - even if this tab is now hidden and no longer active. This is confusing when trying to target CSS as a the selector "ul.tabs-content > li.active" , may not be visible.

Also removed unused tabsContent variable.
This commit is contained in:
James Aitken 2011-06-08 13:42:36 +01:00
parent 0eed0f8949
commit 8b05d2bb35

View File

@ -12,8 +12,7 @@ $(document).ready(function() {
/* Tabs Activiation /* Tabs Activiation
================================================== */ ================================================== */
var tabs = $('ul.tabs'), var tabs = $('ul.tabs');
tabsContent = $('ul.tabs-content');
tabs.each(function(i) { tabs.each(function(i) {
//Get all tabs //Get all tabs
@ -32,8 +31,8 @@ $(document).ready(function() {
tab.removeClass('active'); tab.removeClass('active');
$(this).addClass('active'); $(this).addClass('active');
//Show Tab Content //Show Tab Content & add active class
$(contentLocation).show().siblings().hide(); $(contentLocation).show().addClass('active').siblings().hide().removeClass('active');
} }
}); });