smallfishjs / smallfish

An out-of-the-box React framework based on umi.
https://smallfishjs.github.io/
51 stars 5 forks source link

add umi-plugin-antd #2

Closed nikogu closed 5 years ago

sorrycc commented 5 years ago

https://github.com/umijs/umi/blob/master/packages/umi-plugin-react/src/plugins/antd.js 吧,不满足需求的场景直接提到这里,比如加个 framework: 'smallfish' 的参数。

sorrycc commented 5 years ago

script 也是,相同的功能没必要写两份。

yutingzhao1991 commented 5 years ago

script 也是,相同的功能没必要写两份。

+1 https://github.com/umijs/umi/pull/2416 这个发了之后 bigfish 也会直接替换为 umi 的。

JIACHENG9 commented 5 years ago

根据之前的讨论 plugin 都是最低粒度的,应该是有 umi-plugin-antd 的插件,而不是 umi-plugin-react 里包括很多,要是那样就是之前的 umi-plugin-bigfish 的方式了。 应该是有 umi-plugin-antd,smallfish 使用这个,umi-plugin-react 也可使用这个(其实根据讨论就不应该有 umi-plugin-react,bigfish 之后也会是依赖 umi-plugin-antd)

sorrycc commented 5 years ago

umi-plugin-react 先引用,后面看情况拆吧。

JIACHENG9 commented 5 years ago

那就是 umi-plugin-antd 依赖 https://github.com/umijs/umi/blob/master/packages/umi-plugin-react/src/plugins/antd.js

sorrycc commented 5 years ago

我觉得不用着急拆吧,已有的插件直接引用,比如 umi-plugin-react/lib/antd,然后目前外面没有的可以抽出来。

yutingzhao1991 commented 5 years ago

我觉得可以把 umi-plugin-react/script 理解为一个插件,至于在不在 react 这个插件集里面可以再拆,感觉对于目前来说没有这么重要。