Use .is('tr') instead of .attr('tagName') == 'TR' to determine table rows correctly. about django-dynamic-formset HOT 7 CLOSED

From stan.madueke on November 15, 2011 22:39:57

Status: Accepted
Owner: stan.madueke

from django-dynamic-formset.

From [email protected] on December 22, 2011 03:58:51

I'm using Firefox and I still cannot get this condition to pickup correctly.

The documentation for jQuery states that ".attr() should not be used on arrays", however this appears to be precisely what is happening at this part of the code. I originally thought that I'd done something wrong in the initialisation, but I've been playing with it for the past 2 hours without success.

Changing to ".is('tr')" suffers from the same problem.

from django-dynamic-formset.

From [email protected] on December 22, 2011 04:51:40

Sorry my fault, $$.is('TR') does work in this case after all.

from django-dynamic-formset.

From [email protected] on March 27, 2012 23:47:07

As per http://bugs.jquery.com/ticket/9228#comment:3 , tagName cannot be fetched using the attr function as it is a property not an attribute.

Use $$.prop('tagName') instead of $$.attr('tagName').

from django-dynamic-formset.

From stan.madueke on March 27, 2012 23:53:24

@iakbar: thanks for the link. I think $$.is(...) is the right way to do this though.

from django-dynamic-formset.

From [email protected] on February 21, 2014 14:41:39

I've made the change in my copy of the code to use $$.prop('tagName') and it is working happily now.

from django-dynamic-formset.

Fixed in 1d57afe.

from django-dynamic-formset.