ant-design / antd-style

css-in-js library with antd v5 token system
https://ant-design.github.io/antd-style/
MIT License
219 stars 35 forks source link

🐛[BUG] createInstance创建的StyleProvider的props类型不对 #50

Closed basala closed 1 year ago

basala commented 1 year ago

🐛 bug 描述

使用createInstance创建的styleProvider的props类型不对

📷 复现步骤

import { legacyLogicalPropertiesTransformer } from "@ant-design/cssinjs";

<StyleProvider transformers={[legacyLogicalPropertiesTransformer]}}></StyleProvider>

transformers提示类型不匹配

image

🏞 期望结果

不会提示类型错误

💻 复现代码

可复现 demo

© 版本信息

🚑 其他信息

basala commented 1 year ago
image

ant-design/cssinjs types路径是lib

image

这里是es

导致类型对不上了

arvinxx commented 1 year ago
image

ant-design/cssinjs types路径是lib image 这里是es

导致类型对不上了

antd 的 cssinjs 有导出这个Context的类型么?我记得好像是因为没导出,所以我引了es

basala commented 1 year ago
image

ant-design/cssinjs types路径是lib image 这里是es 导致类型对不上了

antd 的 cssinjs 有导出这个Context的类型么?我记得好像是因为没导出,所以我引了es

看了下没导出,从antd的cssinjs import legacyLogicalPropertiesTransformer的时候路径是lib,应该是这个导致的对不上, 或者能不能改成用React.ComponentProps<typeof StyleProvider>获取下类型

arvinxx commented 1 year ago

从antd的cssinjs import legacyLogicalPropertiesTransformer的时候路径是lib

@basala 你改成从 es 引入呢?

basala commented 1 year ago

从antd的cssinjs import legacyLogicalPropertiesTransformer的时候路径是lib

@basala 你改成从 es 引入呢?

image

改成这样确实不报错了

但是ant-design/cssinjs默认导入路径是lib,下面这样写就会报错

image
arvinxx commented 1 year ago

那感觉只要把包的前面那个 es 改成 lib 就好了?有没有兴趣来个 pr ?

basala commented 1 year ago

那感觉只要把包的前面那个 es 改成 lib 就好了?有没有兴趣来个 pr ?

🉑️,我改了在麻烦大佬看看

basala commented 1 year ago

好像没有合并权限,麻烦大佬看下没问题的话帮忙合并发布一下~

github-actions[bot] commented 1 year ago

:tada: This issue has been resolved in version 3.1.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

arvinxx commented 1 year ago

@basala 来试试

basala commented 1 year ago

@basala 来试试

可以了,感谢大佬