-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider an IE8+ version of HTML5Shiv #70
Comments
I think it's worth exploring. also it's be "IE8" instead of "IE8+" right? |
True that. IE8. (Unless there's something IE9 isn't telling us, or the unlikely event that more polyfilling and style shimmings make their way in.) |
I also thought about using defineProperty to fix IE8. From my point of view this approach is soooo much cleaner and looks extremly promissing. I have just added a first draft in the defineProperty branch. What we need is a really good cloneNode patch, new unit tests, perf (and size) optimizations and documentation. Some important points:
|
Hey guys, pinging this conversation again. What would you think about a rewrite for v4? Here are three suggestions.
|
In short: YES!!! With 2. I would wait a little bit. |
There could be lots of optimisations done for an IE8+ version.
http://www.jonathantneal.com/blog/a-close-shiv/
The text was updated successfully, but these errors were encountered: