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

antd-mobile-samples例子升级到1.0.0之后图标不显示 #840

Closed
vnicers opened this issue Feb 20, 2017 · 17 comments
Closed

antd-mobile-samples例子升级到1.0.0之后图标不显示 #840

vnicers opened this issue Feb 20, 2017 · 17 comments
Assignees
Labels

Comments

@vnicers
Copy link

vnicers commented Feb 20, 2017

本地环境

win7 32

  • antd-mobile 版本:1.0.0
  • 浏览器 (或标明是 react-native) 及其版本:chrome
  • 运行环境及其版本:

你做了什么?

我想试下svg图标的使用,
下载 https://github.com/ant-design/antd-mobile-samples/tree/master/web-webpack
中的代码,里面已经有配置svg的loader。运行成功一切正常,发现是iconfont 图标,
然后修改了antd-mobile的版本到"antd-mobile": "^1.0.0", 跟新后运行,出现警告
warning

浏览器里面运行发现图标不出现。不知道还需要其他什么配置。
demo

你期待的结果是:

能够正常显示svg图标

可重现的在线演示

https://github.com/ant-design/antd-mobile-samples/tree/master/web-webpack
将antd-mobile更新到1.0.0就可重现

@paranoidjk
Copy link
Contributor

@warmhug sample要不同步更新下?

@warmhug
Copy link
Contributor

warmhug commented Feb 20, 2017

sample 代码没问题。可能是 windows 环境的问题,尝试以下方法:

@vnicers
Copy link
Author

vnicers commented Feb 20, 2017

@warmhug 确实是Windows 的问题。刚在Mac 上相同的版本和配置 是没问题的。
像这样的问题你们会处理吗?毕竟windows上开发的人还是很多的

@warmhug
Copy link
Contributor

warmhug commented Feb 20, 2017

以上说的方法,尝试了吗,都解决不了吗?

@jozhi
Copy link

jozhi commented Feb 20, 2017

我 mac 版本也有这样子的问题,跑的是官方的antd-mobile的 nav 例子,暂时降级到0.9解决。

@paranoidjk
Copy link
Contributor

@vnice 请参考这个方法试一下 ant-design/antd-mobile-samples@e8b1d90

@vnicers
Copy link
Author

vnicers commented Feb 21, 2017

@paranoidjk windows上还是不行。刚刚我删除了cnpm 安装的依赖,用npm安装 还是不行,修改了glob 的匹配表达式也不行。依然报警告 就是这个issue // JetBrains/svg-sprite-loader#65
然后 然同事的一台干净的windows机器安装nodejs 下载web-webpack 这个sample 运行后也是没有图标

@warmhug
Copy link
Contributor

warmhug commented Feb 21, 2017

正反斜线的区别?windows 上的路径分隔符都是反斜线,修改下试试看

@vnicers
Copy link
Author

vnicers commented Feb 21, 2017

@paranoidjk @warmhug @jozhi

{ 
       test: /\.(svg)$/i,
       loader: 'svg-sprite',
       include: [path.resolve(__dirname, 'node_modules/antd-mobile/lib')]
 }

这样是可以的。路径的问题早该想到用path.resolve,但是第二个参数不能用正则,如果用cnpm的话这个地址node_modules/antd-mobile/lib就要变了,根据自己情况写。

@vnicers vnicers closed this as completed Feb 21, 2017
@paranoidjk
Copy link
Contributor

@vnice thanks,因为我们这边没有windows机器,稍后会补充到文档里

@paranoidjk
Copy link
Contributor

paranoidjk commented Feb 22, 2017

svg icon 的使用文档已更新,进行了配置的简化和尽可能兼容到windows以及cnpm等各种情况

https://mobile.ant.design/components/icon

ref #866

@jozhi
Copy link

jozhi commented Feb 22, 2017

@vnice 依然不知道怎么使用你发出来的那段代码,配置到哪里? 能不能给出完整的文件路径和内容。。
我这边儿使用的 dva,一些目录结构可能会有些不同,不太清楚从哪下手改。

@paranoidjk
Copy link
Contributor

@jozhi 你是指你用的 roadhog ?

@paranoidjk
Copy link
Contributor

@jozhi

  • 我文档可以再加一下roadhog的配置
  • 但是文档已经说明了如何配置 webpack-loader,在这之上的各种方案的其配置原理都是一样的, 这种问题通过看roadhog的官方文档完全能搞定,它也是基于webpack封装的,且仍可以接受原有的配置方式
  • 社区可能有各种不同的构建方案和环境,我们不可能覆盖全,请谅解。

please try to understand how it work

@jozhi
Copy link

jozhi commented Feb 22, 2017

@paranoidjk 好的 多谢

结合@vnice 老兄的代码改了一下 roadhog 下 wabpack.config.dev.js 文件的配置,可以看到图标了,但是如何配置自定义的图标还要继续研究
config

我这边roadhog版本 "roadhog": "^0.5.2"

@paranoidjk
Copy link
Contributor

@jozhi

@paranoidjk
Copy link
Contributor

@jozhi svg icon 的文档已更新,添加了 roadhog 的支持 https://mobile.ant.design/components/icon

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

No branches or pull requests

4 participants