Tencent / tdesign-vue-next

A Vue3.x UI components lib for TDesign.
https://tdesign.tencent.com/vue-next
MIT License
1.35k stars 445 forks source link

fix(select): fix Option in OptionGroup must be render in v-for #4318

Closed 1379255913 closed 1 week ago

1379255913 commented 1 week ago

🤔 这个 PR 的性质是?

🔗 相关 Issue

💡 需求背景和解决方案

通过group获取option时没有使用getChildComponentSlots来进行获取,而是默认认为是一个数组,没有处理不同的情况,导致发生问题,已经修改为getChildComponentSlots

📝 更新日志

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项。⚠️

github-actions[bot] commented 1 week ago

完成

uyarn commented 1 week ago

fix lint

1379255913 commented 1 week ago

已修复

KawaiiZapic commented 1 week ago

测试用例?

1379255913 commented 1 week ago

这里的测试用例指的是添加单元测试还是上传一下修改后的演示

KawaiiZapic commented 1 week ago

这里的测试用例指的是添加单元测试还是上传一下修改后的演示

单元测试, 说实话我觉得这种bug就不应该没被测试到, 单元测试有些太特化了.

1379255913 commented 1 week ago

单元测试已添加

uyarn commented 1 week ago

这里的测试用例指的是添加单元测试还是上传一下修改后的演示

单元测试, 说实话我觉得这种bug就不应该没被测试到, 单元测试有些太特化了.

谢谢指出问题,确实单测写了却没有测出这么普通的情况是有问题的。