Hello experts.
I'm trying to simplify my jquery code.
This is my second question .
Can i simplify the code below? f.e use index instead of numbers.
// JavaScript Document$(document).ready(function () { //Default Action $(".tab_content1,.tab_content2").hide(); //Hide all content $("ul.tabs1 li:first,ul.tabs2 li:first").addClass("active").show(); //Activate first tab $(".tab_content1:first,.tab_content2:first").show(); //Show first tab content //On Click Event $("ul.tabs1 li").click(function () { $("ul.tabs1 li").removeClass("active"); //Remove any "active" class $(this).addClass("active"); //Add "active" class to selected tab $(".tab_content1").hide(); //Hide all tab content var activeTab = $(this).find("a").attr("href"); //Find the rel attribute value to identify the active tab + content $(activeTab).fadeIn(); //Fade in the active content return false; }); $("ul.tabs2 li").click(function () { $("ul.tabs2 li").removeClass("active"); //Remove any "active" class $(this).addClass("active"); //Add "active" class to selected tab $(".tab_content2").hide(); //Hide all tab content var activeTab = $(this).find("a").attr("href"); //Find the rel attribute value to identify the active tab + content $(activeTab).fadeIn(); //Fade in the active content return false; });});
I am the the first to tell people not to use jQuery for simple tasks. In this case I do not agree with you at all Sybe. Much easier using jQuery than plain JS here.
Thank you for your post.
Attached html and css .Can you take a look.
Open in new window