-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Symbol is not a supported type in Ember.typeOf(); returns "object" #11673
Comments
Is that a real es6 symbol or a polyfiled one |
Most likely ember.js/packages/ember-runtime/lib/utils.js Line 108 in 41be946
|
Hrm...the polyfilled one |
Aww man @rwjblue I looked for an issue that might have already addressed this and I missed it. Sorry about that. |
embers or some other? There are potentially many |
@stefanpenner Whatever the default babel configuration is with Ember CLI. I have not made any modifications and am assuming that http://babeljs.io/docs/usage/polyfill/ is in play but perhaps there is a gap in my understanding. |
im not sure the polyfil decorates itself correctly |
let me check |
perfect, seems like it does
|
We have decided to freeze We can likely revisit at some future point in time, when the landscape has become reasonable. Example insanity:IE11: Object.prototype.toString.call(new Map()) === '[object Object]' Chrome: Version 45.0.2449.0 canary (64-bit)
Chrome: Version 43.0.2357.130 (64-bit)
|
I would have loved to fix this, unfortunately requireing something like core-js to always 100% of time be on the page for a feature to work isn't something we can commit to. Especially for something like Just changing this in embers code-basse actually uncovered several issues. I hope we will revisit in the future. |
If I'm understanding my research correctly for @@toStringTag we will run into the same problem you have listed by using typeof instead, correct? |
nope |
ember.js/packages/ember-runtime/lib/utils.js
The text was updated successfully, but these errors were encountered: