tdesign-vue
tdesign-vue copied to clipboard
fix(tree): 解决 tree 组件指定 keys 属性后,next 版本可能导致 dom 无限刷新的问题
fix #445
🤔 这个 PR 的性质是?
- ☑️ 日常 bug 修复
- [ ] 新特性提交
- [ ] 文档改进
- [ ] 演示代码改进
- [ ] 组件样式/交互改进
- [ ] CI/CD 改进
- [ ] 重构
- [ ] 代码风格优化
- [ ] 测试用例
- [ ] 分支合并
- [ ] 其他
🔗 相关 Issue
https://github.com/Tencent/tdesign-vue-next/issues/445
💡 需求背景和解决方案
📝 更新日志
-
fix(tree): 解决 tree 组件指定 keys 属性后,next 版本可能导致 dom 无限刷新的问题
-
[ ] 本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单
⚠️ 请自检并全部勾选全部选项。⚠️
- ☑️ 文档已补充或无须补充
- ☑️ 代码演示已提供或无须提供
- ☑️ TypeScript 定义已补充或无须补充
- ☑️ Changelog 已提供或无须提供
Codecov Report
Merging #837 (bf7a612) into develop (492d477) will increase coverage by
3.68%. The diff coverage is80.46%.
@@ Coverage Diff @@
## develop #837 +/- ##
===========================================
+ Coverage 57.94% 61.62% +3.68%
===========================================
Files 407 643 +236
Lines 15371 15741 +370
Branches 2931 2817 -114
===========================================
+ Hits 8907 9701 +794
+ Misses 6187 5749 -438
- Partials 277 291 +14
| Impacted Files | Coverage Δ | |
|---|---|---|
| src/alert/_example/icon.vue | 100.00% <ø> (ø) |
|
| src/alert/_usage/index.vue | 0.00% <ø> (ø) |
|
| src/anchor/_usage/index.vue | 0.00% <ø> (ø) |
|
| src/avatar/_example/base.vue | 100.00% <ø> (ø) |
|
| src/avatar/_usage/index.vue | 0.00% <ø> (ø) |
|
| src/badge/_usage/index.vue | 0.00% <ø> (ø) |
|
| src/breadcrumb/_example/custom.vue | 100.00% <ø> (ø) |
|
| src/breadcrumb/_example/dropdown.vue | 100.00% <ø> (ø) |
|
| src/button/_example/icon.vue | 100.00% <ø> (ø) |
|
| src/button/_usage/index.vue | 0.00% <ø> (ø) |
|
| ... and 458 more |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
这个PR是不是要提到 https://github.com/Tencent/tdesign-vue-next/pulls 这边来
这个 PR 已经过时了,因为它已经持续 45 天没有任何活动了。 删除 stale 的标签或评论,否则将在 10 天内关闭。
这个 PR 已经过时了,因为它已经持续 45 天没有任何活动了。 删除 stale 的标签或评论,否则将在 10 天内关闭。
@TabSpace tab哥 这期间其他开发者还迭代了拖拽的能力 还需要把这部分重构加进来
@TabSpace tab哥 这期间其他开发者还迭代了拖拽的能力 还需要把这部分重构加进来
看了下,区别太大,没办法直接合并进来。我先清理覆盖了,然后再重新添加拖动能力吧。
非拖拽树,需要加一下判空
