Skip to content
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

Include ES5-sham #21

Closed
hexalys opened this issue Nov 29, 2013 · 1 comment
Closed

Include ES5-sham #21

hexalys opened this issue Nov 29, 2013 · 1 comment

Comments

@hexalys
Copy link

hexalys commented Nov 29, 2013

Any particular reason you did not include any of the ES5-sham?

@shawnbot
Copy link
Owner

shawnbot commented Dec 2, 2013

Two reasons:

  1. Most of the shams don't provide full coverage of the ES5 spec; and
  2. I've never found a compelling reason to use the Object shams in IE8. As long as d3 mostly works, I'm happy.

If you need the Object shams, you should be fine to just include es5-sham.js alongside aight.

@shawnbot shawnbot closed this as completed Dec 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants