Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: use babel-preset-es2015-ie to adapt ios8.x、Android4.x.. #216

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HansJiang
Copy link

throung our monitoring,it doesn't work in ios8.x、Android4.x and the low version of web browser engines(eg:UC).

BABEL transforming ES6 classes to ES5 syntax has compatibility issues.

Links: same problemlooseMode

It's a better way to use babel-preset-es2015-ie.

differences between babel-preset-es2015-ie and babel-preset-es2015

 [
  [require('babel-plugin-transform-es2015-classes'), {loose: true}],
  require('babel-plugin-transform-proto-to-assign'),
]

@broven
Copy link

broven commented Nov 9, 2018

@SanichKotikov same problem I met, this PR look good to me

@SanichKotikov
Copy link
Collaborator

@broven @HansJiang maybe it's better to use this #243 ?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants