Polyfill does not replace classList.toggle in IE11

[schreifels]

Unexpected behavior

In IE11, classList.toggle does not take a second parameter, and the polyfill does not correct this. As an example, I created http://jsfiddle.net/4w949/64 , which contains the source code from yola/classlist-polyfill, along with a console.log(document.body.classList.toggle); below it.

In this case, [ native code ] is shown in the developer console in IE11.

Expected behavior

For comparison, I took the latest polyfill on MDN in the same fashion: http://jsfiddle.net/4w949/65 . Here, the console.log does not return [ native code ]; it (correctly) returns the implementation from the polyfill, which supports the second parameter.

[schreifels]

Indeed, this is apparently a long-standing issue, see eligrey/classList.js#44

Perhaps we could change the upstream of this module to be https://github.com/stevenvachon/classList.js , which contains a fix?

[snitch]

@beck: This issue is now a priority.

[stevenvachon]

I didn't know about this library. If you merge my fork with this one, I can remove mine -- I'd prefer not to keep it anyway.

[beck]

This is a duplicate of: eligrey/classList.js#54

I've started a new repo that is a proper github branch of the source. I've opened a pull upstream:
eligrey/classList.js#57

If merged, I'll merge down into our branch and cut a new release from https://github.com/yola/classlist-polyfill