22

Please help.

Why is the jquery addClass() not working with event.target? I have made a code and it supposed to add class on the target when clicked, but it does not work, and it says,e.target.addClass is not a function.

http://jsfiddle.net/Lq9G4/

CODE:

<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Class Test</title>
    <script src="http://code.jquery.com/jquery-1.11.0.min.js"></script>
    <style>
        .big {
            font-size: 5em;
        }
        .small {
            font-size: 1em;
        }
    </style>

</head>
<body>
    <p>This is the text</p>
    <script>
        $(function() {
            $("p").click(function(e) {  
                      $("a").removeClass("big").addClass("small");
                      $("this").addClass("big"); //This doesn't work
                      e.target.addClass("big"); //nor this one                
                    });
        });

    </script>
</body>
</html>
Chetan Gawai
  • 2,361
  • 1
  • 25
  • 36
qtgye
  • 3,580
  • 2
  • 15
  • 30

7 Answers7

46

Basically e.target will be a javascript object, you have to convert it into a Jquery object before utilizing its functions like .addClass()

Try,

$(e.target).addClass("big"); 

and at the same time, $("this").addClass("big"); this code will not work since you are passing this as a string. this also a javascript object, you need to convert that too as a jquery object like $(this)

Rajaprabhu Aravindasamy
  • 66,513
  • 17
  • 101
  • 130
3

You have two problem:

$(this).addClass("big"); //Unquote to "this"
$(e.target).addClass("big"); // select e.target with $() wrapper.
Bhojendra Rauniyar
  • 83,432
  • 35
  • 168
  • 231
2

Change this line

$("this").addClass("big"); //This doesn't work

to

$(this).addClass("big"); //This will work work

And further if you need it with the event itself then you can use

$(e.target).addClass('big');
GautamD31
  • 28,552
  • 10
  • 64
  • 85
2

Since .addClass() is a jQuery method, you need a jQuery object, so convert e.target to jQuery object by wrapping it inside $:

$(e.target).addClass("big"); 

Beside that, the other solution is to use $(this). you don't need to wrap this inside " ":

$(this).addClass("big");

Updated Fiddle

Felix
  • 37,892
  • 8
  • 43
  • 55
2
<script>
    $(function() {
        $("p").click(function(e) {
                  $("a").removeClass("big").addClass("small");
                  $(this).addClass("big"); //This doesn't work               
                });
    });

</script>

"this"=>this

2
    e.currentTarget.classList.add("loading_arrow");

you can use above line to add loading_arrow class using js mouse event.

1

Try this:

 $(e.target).addClass('big');

Demo

Milind Anantwar
  • 81,290
  • 25
  • 94
  • 125