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

notice-bar 微信适配的 icon 插槽丢失 #1194

Closed
1 of 5 tasks
codesyofo opened this issue Jun 4, 2024 · 8 comments · Fixed by #1196
Closed
1 of 5 tasks

notice-bar 微信适配的 icon 插槽丢失 #1194

codesyofo opened this issue Jun 4, 2024 · 8 comments · Fixed by #1196

Comments

@codesyofo
Copy link

组件

基础库版本

  • 1.x
  • 2.x

设备机型

手机系统

  • iOS
  • Android
  • HarmonyOS

APP

组件库版本

预期行为

复现步骤

复现 demo

@rayhomie
Copy link
Collaborator

rayhomie commented Jun 5, 2024

微信不支持默认插槽。我这边看能否通过一个属性来控制,兼容你微信的这个场景

@rayhomie rayhomie linked a pull request Jun 5, 2024 that will close this issue
@codesyofo
Copy link
Author

微信不支持默认插槽。我这边看能否通过一个属性来控制,兼容你微信的这个场景

可以对原有的字段 icon 的类型做处理吧,比如设置成 boolean,没必新加个字段
这样可否

@rayhomie
Copy link
Collaborator

rayhomie commented Jun 5, 2024

微信不支持默认插槽。我这边看能否通过一个属性来控制,兼容你微信的这个场景

可以对原有的字段 icon 的类型做处理吧,比如设置成 boolean,没必新加个字段 这样可否

不能出现break change为了微信版本的插槽支持,而去影响支付宝版本的用法

@codesyofo
Copy link
Author

微信不支持默认插槽。我这边看能否通过一个属性来控制,兼容你微信的这个场景

可以对原有的字段 icon 的类型做处理吧,比如设置成 boolean,没必新加个字段 这样可否

不能出现break change为了微信版本的插槽支持,而去影响支付宝版本的用法

icon 属性支付宝版本是 string,微信版本新增 boolean

@rayhomie
Copy link
Collaborator

rayhomie commented Jun 5, 2024

要不辛苦你帮忙提个pr,我看看

@codesyofo
Copy link
Author

另外问下项目用 pnpm 启动不了吗?我用 pnpm dev报错
image

要不辛苦你帮忙提个pr,我看看

@rayhomie
Copy link
Collaborator

rayhomie commented Jun 5, 2024

另外问下项目用 pnpm 启动不了吗?我用 pnpm dev报错 image

要不辛苦你帮忙提个pr,我看看
你先本地 pnpm add @types/node -D一下,可以解决,我后续把这个依赖添加上去

@codesyofo
Copy link
Author

codesyofo commented Jun 5, 2024

@rayhomie 装上后只编译了支付宝的
image
image

node 切回 20可以了,但执行 pnpm dev 还是只编译了支付宝没有微信的

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 a pull request may close this issue.

2 participants