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

[t-select] 限制多选选择器的最大可选数目、限制后不能反向点选取消 #1819

Closed
yaogengzhu opened this issue Nov 22, 2022 · 3 comments
Closed

[t-select] 限制多选选择器的最大可选数目、限制后不能反向点选取消 #1819

yaogengzhu opened this issue Nov 22, 2022 · 3 comments
Labels
to be published fixed, not be published

Comments

@yaogengzhu
Copy link
Contributor

yaogengzhu commented Nov 22, 2022 •
edited
Loading

tdesign-vue 版本

0.50.0

重现链接

https://codesandbox.io/s/6i9b8n?file=/src/demo.vue:14-22

重现步骤

image

No response

期望结果

希望点选可以取消选中

实际结果

无法取消多选

框架版本

vue.2x

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response
@github-actions
Copy link
Contributor

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

yaogengzhu added a commit to yaogengzhu/tdesign-vue that referenced this issue Nov 22, 2022
@yaogengzhu
fix(select): 限制数量后不支持反向选择
bcf5fb7 
fix Tencent#1819
@yaogengzhu yaogengzhu mentioned this issue Nov 22, 2022
Closed
16 tasks
@skytt skytt mentioned this issue Nov 23, 2022
Merged
16 tasks
@skytt
Copy link
Contributor

skytt commented Nov 23, 2022

@yaogengzhu 感谢反馈和 pr，这里我整体梳理精简了一下 option 内部的 disable 相关逻辑，尽量减少判断的分支和代码阅读的成本了

@uyarn uyarn added the to be published fixed, not be published label Nov 23, 2022
@github-actions github-actions bot mentioned this issue Nov 24, 2022
Merged
@xiaosansiji
Copy link
Collaborator

已在 0.50.1 版本修复

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 fixed, not be published
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@skytt @xiaosansiji @uyarn @yaogengzhu