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

使用umijs #769

Closed
wants to merge 1 commit into from
Closed

使用umijs #769

wants to merge 1 commit into from

Conversation

xiaohuoni
Copy link

使用umi,其他东西保留

@superlbr
Copy link
Collaborator

superlbr commented Mar 23, 2018

  1. 先撒花,mark。
  2. umi目前还不认为是稳定成熟的版本,1.2再看

@superlbr
Copy link
Collaborator

superlbr commented Mar 23, 2018

建议:

  1. echarts 版本低了
  2. /UIElement/iconfont报错
  3. antd table: getBodyWrapper is deprecated
  4. models下面的common感觉保留比较好
  5. src/pages/.umi需要添加到.gitignore里

测试Umi Version是1.1.1
Umi优点:

  1. 配置简单
  2. router和model引入都是自动的

Umi缺点:

  1. dev的速度有点慢,页面打开Compiling要等一会,至少1s以上延迟
  2. build速度没有roadhog快, roadhog 63.13s, umi 269.62s
  3. 看gzip,roadhog 打包完1.915MB, index.js是506KB。umi是 9.72MB(含webapp支持,仅计算js/css), umi.js是364.21KB
  4. umi按page切分,切出来小文件数量更多了

@xiaohuoni
Copy link
Author

我只是简单的做了一下整理,还有很多bug和功能没有修改。给云谦看过了,他说他后面会提一个PR。UMI从开始写,我就让我们公司开始用的,没有多少坑,整体还是很优质的

@sorrycc sorrycc mentioned this pull request Mar 26, 2018
@superlbr
Copy link
Collaborator

Duplicate #770

@superlbr superlbr closed this Mar 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants