Target.attr('class') is undefined in mozilla

Question

I have a Strange issue with the latest Mozilla Browser 13.0 with Ubuntu. The code works perfectly in other browsers and also previous versions of mozilla.

Here is the Code

$(document).click(function(event) {
  var target = $(event.target);

  if (!target.attr('class').match(/^RefineClick/) && target.parents('.RefineClick').length == 0) {
   jQuery('.RefineClick').fadeOut();
  }
});

And iam getting the following Error : Target.attr('class').... is undefined

Here is the Screenshot for that: http://i47.tinypic.com/dqoits.png

Why are you selecting this way instead of using jQuery's selectors? — Madara's Ghost, Jun 12 '12 at 14:46
How about using `hasClass` method instead of that wonderful code with the regular expression? — VisioN, Jun 12 '12 at 14:49
Try `var target = $(event.target || e.srcElement);`. @VisioN i think `hasClass` isn't accurate when there are multiple classes involved, correct me if i'm wrong. — Sang Suantak, Jun 12 '12 at 14:52
Except `.hasClass` won't match RefineClick1 for example, where his match method will. — Madara's Ghost, Jun 12 '12 at 14:54
Can you please rewrite the code as you told. because i'am new to jquery. I need was when a popup comes and user clicks any where in the document, the popup should fadeout. — Surjith S M, Jun 12 '12 at 14:55
http://stackoverflow.com/questions/1925276/jquery-classname-woes — zod, Jun 12 '12 at 15:01
http://thicksliced.blogspot.com/2006/12/elementgetattributeclass.html — zod, Jun 12 '12 at 15:01

score 0 · Answer 1 · answered Jun 12 '12 at 14:54

0

Well... working for me?

http://jsfiddle.net/acrashik/b9MCj/

answered Jun 12 '12 at 14:54

Sergei Zahharenko

1,514
12
16

can u workout with my code ? I need was when a popup comes and user clicks any where in the document, the popup should fadeout. – Surjith S M Jun 12 '12 at 15:01
kind of that... [http://jsfiddle.net/acrashik/b9MCj/1/](http://jsfiddle.net/acrashik/b9MCj/1/) – Sergei Zahharenko Jun 12 '12 at 15:08
cant see original code to handle parents class :) not really clear what and when you'd like to hide stuff – Sergei Zahharenko Jun 12 '12 at 15:09

score 0 · Answer 2 · answered Jun 12 '12 at 15:10

0

If there is no class attached to the clicked element target.attr('class') returns undefined (since v1.6); in FF that's expressed as the error you see when you attempt to call .match. (In chrome it would be "can't call method match of undefined")

Add a check for an absent class; if (typeof target.attr('class') === "undefined") return;

answered Jun 12 '12 at 15:10

Alex K.

171,639
30
264
288

if (target.attr('class')) wont pass as well... and looks better :) – Sergei Zahharenko Jun 12 '12 at 15:12

score 0 · Answer 3 · answered Jul 16 '12 at 08:48

You are binding that function to a 'click' event on the entire document. Therefore any click would trigger the handler, including those on elements without a class attribute set. While this would work fine if you tested it only on elements with a defined 'class' attribute (in any browser) it would throw the error (in any browser) because you try and call the method .match() on an undefined value.

"As of jQuery 1.6, the .attr() method returns undefined for attributes that have not been set." API ref.

Since you are using jQuery, who not use it's full capabilities? It has a built in .hasClass() method:

$(document).click(function(event) {
    var target = $(event.target);

    if (!target.hasClass('RefineClick') && target.parents('.RefineClick').length == 0) {
        $('.RefineClick').fadeOut();
    }
});

Note that the input to that method is the class-name rather than a class selector.

Target.attr('class') is undefined in mozilla

3 Answers3