CODEWITHSUNDEEP
javascriptjquery
Courtney Stephenson
Courtney Stephenson

Reputation: 930

Combining jQuery/JavaScript functions

Is there any way to combine all of this to reduce the amount of javascript?

    $(document).ready(function() {

    $(".individualImagebox img").bind("click", function()
    {
        var src = $(this).attr("src");

        if (src.search(/Red\.jpg$/) >= 0) return;

        // Removes the red overlay from the images folder
        $('.individualImagebox img').attr( "src", function () {  
            var thisSRC = $(this).attr( "src");
            return thisSRC.replace(/Red\.jpg$/, ".jpg");
        });

        // Adds the red overlay from the images folder
        $(this).attr( "src", src.replace(/\.jpg$/, "Red.jpg")  );
    });

});

function ShowHide(index) {

    var itemSelector = ".name:eq(" + index + ")";

    $(".name .bio").fadeOut();
    $(".name").not(itemSelector).fadeOut();

    $(itemSelector).animate({"height": "show"}, { duration: 500 });
    $(itemSelector + " .bio").animate({"height": "show"}, { duration: 500 });

}

Upvotes: 0

Views: 492

Answers (2)

bpierre
bpierre

Reputation: 11447

$(function() { // $(function(){}) is a shortcut of $(document).ready(function(){})

    var $activeImg; // Maintain a reference to the last activated img

    $(".individualImagebox img").click(function(){
        if (!!$activeImg) {
            $activeImg.attr("src", function(i, src){
                return src.replace(/(.+)Red\.jpg$/, "$1.jpg");
            });
        }
        $activeImg = $(this).attr("src", function(i, src){ // replace attribute and updates active img reference
            return src.replace(/(.+)\.jpg$/, "$1Red.jpg");
        });
    });
});

I don’t know exactly what you are trying to do but if possible, you should toggling a class instead of modifying the src attribute.

Upvotes: 1

Andre Pena
Andre Pena

Reputation: 59336

Combining methods won't save you space. Take a look at

http://developer.yahoo.com/yui/compressor/

Upvotes: 1

Related Questions