Are not all ES6 features polyfilled?

[matthijn]

In the documents on the section of polyfills there is the following line: "In addition to ES6 syntax features, it also supports:"

Which makes me presume all ES6 features are supported (and then some). However, when using for-of it seems to fail in IE11 which has no support for for-of.

Below the failure.

Now for that example it's a trivial fix, but the sentence in the documentation makes me think it should be supported. Can't seem to add support without ejecting, which I prefer not to for this simple thing. But still.

I've updated to react-scripts 1.0.10, but that does not seem to make a difference.

[gaearon]

It says:

In addition to ES6 syntax features, it also supports [...]

(Emphasis added)

So it wasn't about runtime ES6 feature, but about syntax.

The paragraph below mentions polyfills:

Note that the project only includes a few ES6 polyfills:

[...]

If you use any other ES6+ features that need runtime support (such as Array.from() or Symbol), make sure you are including the appropriate polyfills manually, or that the browsers you are targeting already support them.

Hope this explains it! You probably missed this part.

Can't seem to add support without ejecting, which I prefer not to for this simple thing. But still.

You don't need to eject. Just add a polyfill like you normally would, in src/index.js before other imports. For example

npm install --save core-js

import 'core-js/es6/symbol';

Does this help?

[matthijn]

It does indeed! I thought "for of" was syntactic sugar. So perhaps that's where the confusion came from.

Thanks a bunch!

[gaearon]

Yea, syntax that depends on runtime features is where it gets tricky. Thanks for highlighting this. We should probably mention such caveats.

[Timer]

Maybe it's worth including a Symbol polyfill? IIRC it'll be needed for React 16 and I would consider adding it simply because it's used by for-of (which is syntax).

[gaearon]

It’s not required by React 16, no (but Map, and Set are).
That said I do want to include Symbol soon.

[viankakrisna]

If #2358 merged, maybe we don't have to curate the polyfills anymore? We can statically analyze it and ship what they need. IMO leaving polyfills to the user causes an issue like this.

I think adding a polyfill counts as "config" on the user space, so, can we add this magic? 😄

[meooxx]

@gaearon how to config?(like object-rest-spread, async-generator ... ) i have failed many times

[Timer]

@meooxx I'm not sure what you're asking? All of these things work out of the box -- you do not need to think about them.

[meooxx]

@Timer .. Note that the project only includes a few ES6 polyfills:

[Timer]

ES6 polyfills enable new language features, not syntax features.
Async functions and object rest spread of syntax features. Go ahead an try 'em!

[meooxx]

@Timer but it throws error " need an appropriate loader to handle this file type "

[meooxx]

@Timer you are right . my mistakes