I was just wondering if someone could give this a quick look and see
if there is anything wrong with it. It seems to be working on the web
site but perhaps there is something I could do to make it more
efficient.
$(document).ready(function(){
//THIS HIDES ALL LISTED DIVS ON LOADING OF PAGE
$('div.blackbook').hide();
$('div.canvas').hide();
$('div.pencil').hide();
$('div.murals').hide();
$('div.shows').hide();
$('div.tattoos').hide();
$('div.miscellaneous').hide();
$('a#blackbook').click(function() {
//HIDE DIVS
hide_divs();
//SHOW LISTED DIV
$('.blackbook').show('fast');
return false;
});
$('a#canvas').click(function() {
//HIDE DIVS
hide_divs();
//SHOW LISTED DIV
$('.canvas').show('fast');
return false;
});
$('a#pencil').click(function() {
//HIDE DIVS
hide_divs();
//SHOW LISTED DIV
$('.pencil').show('fast');
return false;
});
$('a#murals').click(function() {
//HIDE DIVS
hide_divs();
//SHOW LISTED DIV
$('.murals').show('fast');
return false;
});
$('a#shows').click(function() {
//HIDE DIVS
hide_divs();
//SHOW LISTED DIV
$('.shows').show('fast');
return false;
});
$('a#tattoos').click(function() {
//HIDE DIVS
hide_divs();
//SHOW LISTED DIV
$('.tattoos').show('fast');
return false;
});
$('a#miscellaneous').click(function() {
//HIDE DIVS
hide_divs();
//SHOW LISTED DIV
$('.miscellaneous').show('fast');
return false;
});
});
//HIDES ANY VISIBLE DIVS
hide_divs = function() {
//HIDE ANY VISIBLE DIVS BEFORE CONTINUING
if ($('div.main').is(':visible')) {
$('div.main').hide('fast');
}
if ($('div.blackbook').is(':visible')) {
$('div.blackbook').hide('fast');
}
if ($('div.canvas').is(':visible')) {
$('div.canvas').hide('fast');
}
if ($('div.pencil').is(':visible')) {
$('div.pencil').hide('fast');
}
if ($('div.murals').is(':visible')) {
$('div.murals').hide('fast');
}
if ($('div.shows').is(':visible')) {
$('div.shows').hide('fast');
}
if ($('div.tattoos').is(':visible')) {
$('div.tattoos').hide('fast');
}
if ($('div.miscellaneous').is(':visible')) {
$('div.miscellaneous').hide('fast');
}
}
Any comments would be appreciated.