-
Notifications
You must be signed in to change notification settings - Fork 239
Does not work with IE8 #46
Comments
Doubtful, Internet Explorer is this library's weak point. |
As I can see the problems are in |
I think we should ask @suprememoocow, he spent a lot of time rewriting it to make it performant, so I'd hate to lay waste to his work. |
You can use something like this to make it work https://github.com/es-shims/es5-shim |
Oooh, that'd actually be quite cool. |
It seems that es5-shim is the simplest way to go. It is better than update the library. @hassankhan, may be something about es5-shim and IE compatibility should be noticed in the readme? |
Hey folks, Sorry I've been so quiet. Been really busy. es5-shim is a great idea. If we wanted to reduce the dependencies, it would also be really simple to build our own shims, for example:
I guess it depends on how many people require IE8 support - if it's only a few, then perhaps using es5-shim would be the best option. |
@JustBlackBird Well, we already have a Testling banner that shows support across different browsers. Is that what you meant? In general, I'm 👍 on this, if it isn't too much work. |
@hassankhan I mean a hint in the readme with notice about es5-shim. For example it can be something like the following:
I'm not sure if it's needed but it can save time for developers who care about old browser compatibility. |
Yeah, the idea is that the shims are used by the developer using this, not a decision to be made inside the library. There may be other libraries which also rely on es5 features that can be shimmed, but you don't want those shims to be duplicated in every lib |
Hmm ok, but how would we automate testing it if it actually works using the shim i.e. using Testling? |
Then the shim must exist in the tests, it should be included before the library in the html page, or required before the library if using browserify or similar |
I'll be honest with you, I'm not sure where to start with respect to making tests work like that. Do you think it would be possible to make a quick proof-of-concept PR just so I can see how it would work? Once we know for sure that it works with the shims, then I'll update the documentation, too, thanks 😄 |
Ah bollocks to it, if someone wants to run it on IE8, they can do it at their own risk. I'll add a disclaimer. |
The library is not compatible with IE8. Is there any chance the compatibility will be added?
The text was updated successfully, but these errors were encountered: