hasClass always returns true about jqlite HOT 6 CLOSED

After having a look at jqlite.js I believe the code that says (lines 661-663):

          if( this[i].classList.item(className) ) {
              return true;
          }

Should be replaced with something like:

        for( var i2 = 0, len2 = this[i].classList.length; i2 < len2 ; i2++ ) {
          if(this[i].classList.item(i2)==className) {
              return true;
          }
        }

Or am I wrong?

from jqlite.

Fixed in commit 105f8de (new release v0.2.5)

thanks @AleCaste !!

from jqlite.

You're welcome.
I can see you've used the method 'contains' in the fix, but I'm not quite sure if that would work on the client side on all browsers. Would the array method 'indexOf' be a better alternative in this case?

from jqlite.

Hi @AleCaste this implementation is for browsers that support classList, old browsers will use the alternative definition of hasClass see code.

from jqlite.

Ahhh ok. I missed that. Thanks a lot for the clarification.

from jqlite.

ur welcome! thanks you for your feedback!

from jqlite.