Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

Safari 10 compatibility? #123

Closed
mixonic opened this issue Jul 13, 2017 · 4 comments
Closed

Safari 10 compatibility? #123

mixonic opened this issue Jul 13, 2017 · 4 comments

Comments

@mixonic
Copy link
Member

mixonic commented Jul 13, 2017

Refs: 201-created/bodega-glimmer#5

Should uglify be passed the mangle: { safari10: true } option by default?

@bantic
Copy link

bantic commented Jul 13, 2017

There's a little more detail about this in this glimmer-blueprint issue: glimmerjs/glimmer-blueprint#58

@rwjblue
Copy link
Member

rwjblue commented Jul 14, 2017

Yes, we should enable this option by default but we might want to consider adding this to the defaults of ember-cli-uglify until safari10 ages out...

@acorncom
Copy link
Member

Looks like the fix for this has been merged in ember-cli-uglify (ember-cli/ember-cli-terser#26), can we close this?

@rwjblue rwjblue closed this as completed Oct 25, 2017
@rwjblue
Copy link
Member

rwjblue commented Oct 25, 2017

Thanks for the write up @bantic, and thanks for the reminder @acorncom!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants