Microsoft Edge not supported about g2 HOT 6 CLOSED

yes ... I agree. I also stumbled yesterday over that bug and intended to raise an issue also,

But there is no need for a new Object method, as Object.assign works identically to the alternate spread syntax.

See this link for a recent discussion of PWA's.

from g2.

Should be polyfilled for Edge now.
If any issues occur please let me know.

from g2.

For example in line 82 ...

 g.beg(Object.assign({}, {x:this.x,y:this.y,lw:1}, this.defaults.style,this.style));

there is no need to have that empty object in the beginning, as the the second one can be modified as destination, as in

 g.beg(Object.assign({x:this.x,y:this.y,lw:1}, this.defaults.style,this.style));

This should be less memory intensive and more performant.

from g2.

I actually do think the difference can't be measured, or the JIT-Compiler would optimize this part anyway.
Anyway i changed this in 6a68bc3

from g2.

Thanks, works now.

Although there is still some spreading going on in g2.core.js (e.g. here or here), it doesn’t seem to be a problem for mecEdit for now.

Maybe you can still get rid of those too, in the future.

from g2.

This should be no problem.
In line 473 a rest parameter is used, which looks like a spread-operator, but is none.
The others are supported in Edge, since they are function-calls and not object-literals.

Let me know in case I missed one.

from g2.