Re: Can you help me make my javascript more efficient

[Steven Gardner]

Thanks Guys!

I’ve edited my stuff and here’s how it looks.

	/* Set up variables */

var filtersBox = $('.filtersbox'),

settingsBox = $('.settingsbox'),

settings = $('.settings'),

filters = $('.filters'),

closeSettings = $('.closesettings'),

closeFilters = $('.closefilters');



settings.click(function(event){

event.preventDefault();

if(settingsBox.is(':hidden')){

settingsBox.slideDown(400).removeClass('hidden');

filtersBox.slideUp('fast').addClass('hidden');

settings.addClass('settingsactive');

closeSettings.slideDown(400);

}else{

settingsBox.slideUp(400).addClass('hidden');

closeSettings.hide(400);

settings.removeClass('settingsactive');

}

});



filters.click(function(event){

event.preventDefault();

if(filtersBox.is(':hidden')){

filtersBox.slideDown(400).removeClass('hidden');

settingsBox.slideUp('fast').addClass('hidden');

settings.removeClass('settingsactive');

filters.addClass('filtersactive');

}else{

filtersBox.slideUp(400).addClass('hidden');

filters.removeClass('filtersactive');

}

});



closeSettings.click(function(event){

event.preventDefault();

settingsBox.slideUp(400).addClass('hidden');

settings.removeClass('settingsactive');

closeSettings.hide(400);

});



closeFilters.click(function(event){

event.preventDefault();

filtersBox.slideUp(400).addClass('hidden');

filters.removeClass('filtersactive');

});