-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
feat: multiple options && activeIconSetPath about cascader comp #6741
base: master
Are you sure you want to change the base?
Conversation
fieldNames?: FieldNamesType | ||
activeIconSetPath?: boolean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
activeIconSetPath 这个命名好奇怪。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
哈哈,因为是从 activeIcon 这个属性派生出来的,然后上面的 tab我个人感觉有点像是路径的意思然后就取的这个名字,看看大佬有什么更好的命名
PR preview has been successfully built and deployed to https://antd-mobile-preview-pr-6741.surge.sh |
fix #6334
移动端 级联选择器支持多选逻辑,一些ts类型和细节需要调整
之所以不直接用这个按钮是因为在mobile可能不好看
并且要处理类似于点击子级然后父级也要被选中的逻辑,由于是分层级的array数组,所以如果添加了这一个设计那么每一次点击这个选项都要深度遍历一次 options,因此将这个按钮和 activeIcon这个 元素做了合并
antd.mobile.mp4