Skip to content
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

MIP 核心错误依赖了组件 polyfill #413

Closed
yenshih opened this issue Dec 12, 2018 · 2 comments
Closed

MIP 核心错误依赖了组件 polyfill #413

yenshih opened this issue Dec 12, 2018 · 2 comments
Assignees
Labels
browser quirks beowser quirks bug Something isn't working has PR

Comments

@yenshih
Copy link
Contributor

yenshih commented Dec 12, 2018

BUG 描述

  • mip-bind 依赖 ES6 特性 Set
  • Setpolyfill 位于 __mipComponentsWebpackHelpers__

复现步骤

  • installMipComponentsPolyfill 移至 mip-bind 注册后执行
  • 根据 phantomjs 环境下报错 #402 复现的 phantomjs 环境报错,低端机应该也会报错

期望结果

  • installMipComponentsPolyfill 仅用于组件依赖
@yenshih yenshih added bug Something isn't working browser quirks beowser quirks labels Dec 12, 2018
@yenshih
Copy link
Contributor Author

yenshih commented Dec 17, 2018

修复方案

  • 移除 mip-bind 中的 Set,移除 Setpolyfill
  • __mipComponentsWebpackHelpers__ 使用更严格的 babel-runtime 白名单 @clark-t
  • 核心 [...arr] 依赖 Array.frompolyfill,需保留
  • 其他 polyfill 可移至组件注册前(至少需回归 phantomjs
  • 在 12.31 后继续推进

@yenshih yenshih added the has PR label Dec 17, 2018
@yenshih yenshih changed the title mip-bind 依赖 ES6 特性 Set MIP 核心错误依赖了组件 polyfill Dec 17, 2018
@yenshih
Copy link
Contributor Author

yenshih commented Jan 10, 2019

关于 __mipComponentsWebpackHelpers__ 的问题在新的 issue 继续跟进

@yenshih yenshih closed this as completed Jan 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser quirks beowser quirks bug Something isn't working has PR
Projects
None yet
Development

No branches or pull requests

1 participant