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.

Sign up for GitHub

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

Jump to bottom

[Table] active-change的参数好像不太对 #4000

Closed
Bananer-fl opened this issue Mar 6, 2024 · 3 comments
Closed

[Table] active-change的参数好像不太对 #4000

Bananer-fl opened this issue Mar 6, 2024 · 3 comments
Labels
to be published to be published

Comments

@Bananer-fl
Copy link

tdesign-vue-next 版本

^1.8.0

重现链接

No response

重现步骤

在多选的情况下取消一行，触发的active-change的ActiveChangeContext的activeRowList是空的无论是否还剩余有选中的行，与activeRowKeys对应不上，设计就是这样的嘛。。。那activeRowList这个命名怕是不太对。。

期望结果

例如多选状态下，选了三行，取消一行触发active-change时，ActiveChangeContext的activeRowList应该还有剩余的两行，与activeRowKeys对应上

实际结果

No response

框架版本

No response

浏览器版本

No response

系统版本

No response

Node版本

v20.4.0

补充说明

No response
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 6, 2024

👋 @Bananer-fl，感谢给 TDesign 提出了 issue。
请根据 issue 模版确保背景信息的完善，我们将调查并尽快回复你。

@XBIsland
Copy link
Contributor

activeRowType 为 multiple 时，取消高亮时会执行 useRowHighlight 中的 handleInactive 函数，该函数将 activeRowList 赋值为 [] ，所以 active-change 时获取到的 activeRowList 永远为 []。

image

XBIsland pushed a commit to XBIsland/tdesign-vue-next that referenced this issue Mar 11, 2024
fix(table): activeRowList assignment error when `activeRowType = tr…
7405f7a 
…ue` (Tencent#4000)
@XBIsland XBIsland mentioned this issue Mar 11, 2024
Merged
16 tasks
chaishi pushed a commit that referenced this issue Mar 11, 2024
@XBIsland
fix(table): activeRowList assignment error when `activeRowType = mu…
c2d6f26 
…ltiple` (#4010)

* docs(table): update table demo

* fix(table): `activeRowList` assignment error when `activeRowType = true` (#4000)

---------

Co-authored-by: 陈煜坚 <chenyujian@oigcn.cn>
@uyarn uyarn added the to be published to be published label Mar 11, 2024
@uyarn uyarn changed the title table active-change的参数好像不太对 [Table] active-change的参数好像不太对 Mar 11, 2024
@uyarn
Copy link
Collaborator

uyarn commented Mar 29, 2024

fixed 1.9.1, thanks for @XBIsland

@uyarn uyarn closed this as completed Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
to be published to be published
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@uyarn @XBIsland @Bananer-fl