Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(table): 修复设置 reserveSelectedRowOnPaginate=false时全选被禁用切换分页后才恢复正常并且无法全选的问题 #4670

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ZhuYuRan001
Copy link

@ZhuYuRan001 ZhuYuRan001 commented Oct 21, 2024

修复设置 reserveSelectedRowOnPaginate=false 时全选被禁用切换分页后才恢复正常并且无法全选的问题

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

#4403

💡 需求背景和解决方案

问题:
image
修复后:
image
原因:
useRowSelect.tsx文件种全选复选框是否禁用的props是一个计算属性,reserveSelectedRowOnPaginate为false时返回currentPaginateData,由于初始化时定义currentPaginateData变量值为data.value,失去了对data的响应式控制,导致后续从远程获取到数据源之后,computed也没有监听到currentPaginateData的变化。
修复方案:
修改定义currentPaginateData为计算属性
关键代码:
image
image
image

📝 更新日志

  • fix(table): 修复设置 reserveSelectedRowOnPaginate=false 时全选被禁用切换分页后才恢复正常并且无法全选的问题

  • 本条 PR 不需要纳入 Changelog

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

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

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

Copy link
Contributor

失败

@uyarn
Copy link
Collaborator

uyarn commented Oct 21, 2024

有赋值的逻辑,这里如果要这样修改需要同步修改赋值的地方

@ZhuYuRan001
Copy link
Author

有赋值的逻辑,这里如果要这样修改需要同步修改赋值的地方

可以描述的更详细一点吗,有点不太明白意思。是指需要同步修改canSelectedRows和其他涉及到currentPaginateData的地方吗

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants