Tencent / tdesign-mobile-vue

A Vue3.x Mobile UI components lib for TDesign.
https://tdesign.tencent.com/mobile-vue
MIT License
249 stars 143 forks source link

[t-dialog] confirmBtn的插槽不生效 #246

Closed Jack-yo closed 1 year ago

Jack-yo commented 1 year ago

tdesign-mobile-vue 版本

0.9.2

重现链接

https://github.com/Tencent/tdesign-mobile-vue/blob/0.9.2/src/popup/popup.vue

重现步骤

t-dialog组件引用的时候,涉及到popup组件,popup组件中使用到overlay,没有传入visible这个必须属性。导致t-dialog的确定按钮插槽无法使用。

期望结果

t-dialog可以传入插槽

实际结果

t-dialog传入插槽不成功

框架版本

No response

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response

github-actions[bot] commented 1 year ago

👋 @Jack-yo,感谢给 TDesign 提出了 issue。 请根据 issue 模版确保背景信息的完善,我们将调查并尽快回复你。

LeeJim commented 1 year ago

我来处理下

LeeJim commented 1 year ago

看了下 confirmBtn 插槽可以正常使用

Jack-yo commented 1 year ago

你的测试demo是?

LeeJim commented 1 year ago

麻烦用 codesandbox 提供下你的问题代码?

Jack-yo commented 1 year ago

https://codesandbox.io/s/wonderful-wozniak-1ertu2?file=/src/components/HelloWorld.vue 这是demo链接。我看了你们的源代码,怀疑是插槽实现的时候,不应该是先判断是否有值再去slot,而是在slot里面判断是否有内联属性才显示默认插槽,否则就需要插入用户的插槽

Jack-yo commented 1 year ago

你可以在我的demo的helloworld组件中的t-dialog内联content="--"属性,然后我写的插槽内容就会展示出来,这个情况正正验证我说的那个问题。

LeeJim commented 1 year ago

好的。我看看

LeeJim commented 1 year ago

确实有问题,我来处理下