tdesign-react icon indicating copy to clipboard operation
tdesign-react copied to clipboard

fix(tree-select): prevent redundant `onChange`, it and valueDisplay parameters undefined

Open RSS1102 opened this issue 9 months ago • 2 comments

🤔 这个 PR 的性质是?

  • [x] 日常 bug 修复
  • [ ] 新特性提交
  • [ ] 文档改进
  • [ ] 演示代码改进
  • [ ] 组件样式/交互改进
  • [ ] CI/CD 改进
  • [ ] 重构
  • [ ] 代码风格优化
  • [ ] 测试用例
  • [ ] 分支合并
  • [ ] 其他

🔗 相关 Issue

💡 需求背景和解决方案

当前点击已选中项时,仍能触发 onChange 事件,且此时 onChangevalueDisplay 参数均为 undefined。 修正和 vue-next 表现一致:

  1. 点击已选中项时不触发 onChange
  2. 单选状态下,每次点击任何值时都更新 valueDisplay 参数。
743068296802

image

📝 更新日志

  • fix(tree-select): 修复点击已选中项时,仍能触发 onChange 事件,且此时传递参数为 undefined

  • fix(tree-select): 修复点击已选中项时,valueDisplay 参数为 undefined

  • [ ] 本条 PR 不需要纳入 Changelog

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

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

  • [x] 文档已补充或无须补充
  • [x] 代码演示已提供或无须提供
  • [x] TypeScript 定义已补充或无须补充
  • [x] Changelog 已提供或无须提供

RSS1102 avatar Mar 27 '25 11:03 RSS1102

Open in Stackblitz

npm i https://pkg.pr.new/tdesign-react@3456

commit: b578b0a

pkg-pr-new[bot] avatar Mar 27 '25 11:03 pkg-pr-new[bot]

完成

github-actions[bot] avatar Mar 27 '25 11:03 github-actions[bot]

duplicate #3573

RSS1102 avatar Sep 11 '25 09:09 RSS1102