Question

I have a link like this :

<a href="#" onclick="renkDegistir()" title="Yaz .......

And JS code :

function renkDegistir()
    {  
        $("#cont #main article").addClass("siyah-stil");
    }

When click this link siyah-stil class is added to article. But i want to do, if link clicked 2nd time remove siyah-stil class.

In summary ,

on first click : addClass("siyah-stil");
on second click : removeClass("siyah-stil");

Answer 1

Try using toggleClass():

function renkDegistir()
{
    $("#cont #main article").toggleClass("siyah-stil");
}

But I’d suggest also removing the in-line onclick attribute, and switching to jQuery’s click() event-handler:

$('a').click(
    function(){
        $("#cont #main article").toggleClass("siyah-stil");
    });

---

Edited to address @Eray’s comment (below):

Can you explain [to] me , why jQuery’s click() function [is] better than in-line onclick attribute?

The reason I prefer to use non-inline code is that it reduces the complexity of changing the onClick events; since they get changed in one place, rather than having to change the onclick attribute in every a (or other) element. Admittedly, you could achieve that benefit by simply changing the renkDegistir() function, but then you end up with functions named for posterity, rather than the inherent nature of the function.

It also makes it easier for others to take over, and adapt your code, and to iron out bugs when they appear.