Doesn't work in IE 8.0.7601.17514 / Win 7 #36
Comments
|
Hmm, odd. I will need to investigate further before removing the condition, but thanks for informing me. Years ago when I first made classList.js, that condition was working fine in IE8. Maybe a security patch changed something. |
|
Agreed, having a condition there is probably a good idea. There may be some other kind of duck typing you can check on the error. |
|
I can also confirm this issue. |
|
👍 Tested in IE8 with XP VM and this is still a problem. The solution proposed works great. |
|
How can you replicate the problem? I have the same IE8 version in a Windows XP VM and I don't have this issue. |
|
Windows XP VM from Microsoft running in VirtualBox on a host Macbook. I use it via require/import it in a browserify/webpack bundle with babel transpiling |
|
I'm running my Windows XP VM (32 bits) in a Virtual Box hosted on a Windows 7 VM (64 bits). First time I hear about transpiling. I simply load classList.js in my |
|
What you had hold of initially was an ActiveX exception (the number is an HRESULT). Clearly IE 11 is no longer using ActiveX objects behind the scenes. Might as well remove the check as it simply causes all other exceptions to fail silently. There's really no way to deal with the exception anyway. What's the point of having it enumerable in some browsers and not in others? Realize this will never be a perfect imitation of A "polyfill" that doesn't actually work as expected only serves to deceive other scripts. It fools their feature detection into thinking that the feature is there, but it's really just an incomplete imposter. In general, DOM "polyfills" are highly ill-advised and this one is a good example of why. Better to write wrappers (e.g. Also note that transpiling is just a red herring here. HTH |
|
I'm super busy with my proprietary stuff and missed this a year ago. I merged #43 just now. I'm sorry about this issue affecting any of you. Please tell me if the changes I have merged have fixed this issue for you. |
|
@eligrey Could you please update the classList.js on the CloudFlare CDN and post the updated URL's in the readme.md? |
|
I use this commit beck@d94a623 but it seems ie11 wont support multiple args. So I have to write: |
I've just tried your polyfill in the above version and it doesn't work. It fails here: https://github.com/eligrey/classList.js/blob/master/classList.js#L174
Turns out the exception thrown isn't what you were expecting. The exception has no
numberproperty. It is aTypeErrorstating that "getters & setters can not be defined on this javascript engine".Removing the condition
if (ex.number === -0x7FF5EC54) { ...fixes the problem.The text was updated successfully, but these errors were encountered: