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

IntlShim and Intl names may be exchanged. #71

Closed
clmath opened this issue Oct 6, 2014 · 1 comment
Closed

IntlShim and Intl names may be exchanged. #71

clmath opened this issue Oct 6, 2014 · 1 comment

Comments

@clmath
Copy link
Member

clmath commented Oct 6, 2014

I think it is more natural to require Intl if you want to pick any available Intl implementation (ie: the one from the browser or the polyfill) and to require IntlShim if you specifically want the polyfill.

@clmath clmath added this to the 1.0.0 milestone Oct 6, 2014
@JCEmmons
Copy link
Member

Perhaps you might be correct, but it also brings into question how much we trust the various browser implementations. One of the reasons we put this package together in the first place was to offer a somewhat complete package of locales ( at least one that would meet all of IBM's requirements ). So in that type of situation, we would want Intl = the polyfill. At least by doing so, we can guarantee consistent behavior across the various browsers.

I'm not inclined to change it at this point without a really good reason.

@cjolif cjolif removed this from the 1.0.0 milestone Oct 30, 2014
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

3 participants