tdesign-react
tdesign-react copied to clipboard
fix(Popconfirm): fixes an issue that default icon still displayed whe…
…n icon for popcontent is null
🤔 这个 PR 的性质是?
- [x] 日常 bug 修复
- [ ] 新特性提交
- [ ] 文档改进
- [ ] 演示代码改进
- [ ] 组件样式/交互改进
- [ ] CI/CD 改进
- [ ] 重构
- [ ] 代码风格优化
- [ ] 测试用例
- [ ] 分支合并
- [ ] 其他
🔗 相关 Issue
无。
💡 需求背景和解决方案
使用 <Popcontent /> 且设置 icon 为 null 时,组件仍会渲染一个图标。这和 confirmBtn, cancelBtn 等其他 prop 的逻辑不一致。
📝 更新日志
-
fix(Popconfirm): 修复
icon={null}时,仍会显示默认图标的问题 -
[x] 本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单
⚠️ 请自检并全部勾选全部选项。⚠️
- [x] 文档已补充或无须补充
- [x] 代码演示已提供或无须提供
- [x] TypeScript 定义已补充或无须补充
- [x] Changelog 已提供或无须提供
不渲染 icon 可以直接传入 <></> 🤔
这个问题感觉是目前 TNode 类型进行渲染的时候没有统一
许多组件,例如 TreeSelect 的 empty 也支持 TNode, 但它想要隐藏,null 也是不生效的,只能 <></>
包括有些 TNode 甚至支持传入 false 就隐藏 (?)
所以感觉这个需要讨论后,再决定是否改
React 端未来应该会参考 Vue Next 的 useContent 进行统一适配自定义节点的渲染
