【BREAK CHANGE】Upgrade svg-sptite-loader

[AvailCat]

Notice

antd-mobile Icon 所依赖的开源 webpack loader svg-sprite-loader 最近从 0.3 直接升级到了 2.0, 其配置方案发生了 break change.

但由于此 loader 的版本由用户代码控制，而功能被 antd-mobile 和用户代码所共同依赖， antd-mobile 团队只能选择在 2.0 对其进行升级，请各位用户在 antd-mobile@1.x 期间暂时不要升级 svg-sprite-loader，在项目里仍然使用 svg-sprite-loader@0.3.x 即可。#1283

---

Version

1.1.2

Environment

svg-sprite-loader@2, unrelated to other environments

Steps to reproduce

1. Use svg-sprite-loader 2.0.0 to
2. Call Toast.fail('Whoops')
3. Reproduced

What is expected?

Show toast

What is actually happening?

Browser throw an exception

---

Variable type in index.web.js with svg-sprite-loader 0.x is string，like "#fail"
It's an object after upgrade to svg-sprite-loader 2.0+, include a default property，with sub property "id", "id" is the real type

[paranoidjk]

right now we haven't upgrade svg-sprite-loader, just use the package.json version
https://github.com/ant-design/ant-design-mobile/blob/master/package.json#L118

[AvailCat]

So you should tell user to install svg-sprite-loader@0.3.1 in docs

[paranoidjk]

nope, you do not need to install svg-sprite-loader by yourself, antd-mobile dependencies included it.

[paranoidjk]

@Meeeeow sorry, my mistake, it's devDeps, will not be install when you npm install antd-mobile

[AvailCat]

Nope, I think devDependencies will not install when package install as a dependency.
I'm using antd-mobile with style: true in babel-plugin-import options, so less file will passed to mine webpack installation

[paranoidjk]

@Meeeeow whould you like to PR to help use upgrade svg-sprite-loader?

[AvailCat]

Yep, maybe tonight

[paranoidjk]

38e4ae2?diff=unified
已经更新了文档，暂时先别升级，这个升级方案要仔细评估能否向下兼容，业务有太多依赖

-npm install svg-sprite-loader -D
+npm install svg-sprite-loader@0.3.1 -D

[paranoidjk]

https://github.com/kisenka/svg-sprite-loader/blob/master/2.0.md

[paranoidjk]

close in favor of #1740