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

树状 开可选时 拓展为单击文字部分甚至行即可实现勾选✔ 而不是只能点击icon #1776

Open
Lee-Lily-Lea opened this issue Mar 8, 2024 · 4 comments
Labels

Comments

@Lee-Lily-Lea
Copy link

image
显示是如上图所示 只能点击icon实现选择

image
开启Hoverable后 有了 "行"的状态 开启Activatable后 有了可点的效果
但是没能实现点击后 加入select

希望在开启选择模式是 激活(Activatable)改为select

@capdiem
Copy link
Contributor

capdiem commented Mar 8, 2024

@Lee-Lily-Lea 此功能我倾向于增加一个属性来开启点击行即可选中的特性。

@Lee-Lily-Lea
Copy link
Author

Lee-Lily-Lea commented Mar 8, 2024

@Lee-Lily-Lea 此功能我倾向于增加一个属性来开启点击行即可选中的特性。

@capdiem

感觉和Activatable重复

实际上当开启了选择模式的时候 就不应该存在 激活项这个概念

Activatable更像是单选 (虽然Activatable有多选模式)

Selectable更像是多选

可以考虑下 类似其他几个组件的模式 改成单选和多选 (Multiple)

Activatable和Selectable更像是在UI的区别
一个是 给row加上 激活(颜色标记) [点亮]
一个是给前置增加 图标 [拾取]

个人倾向这俩是一个枚举
0 啥都不开
1 开启选择图标
2 开启激活模式

然后单独的属性bool Multiple
对于上述两种模式 影响相似 且不冲突

@capdiem
Copy link
Contributor

capdiem commented Mar 8, 2024

@Lee-Lily-Lea 最小改动的方式只能是增加一个属性而不破坏现有的API,也许叫SelectOnRow

@Lee-Lily-Lea
Copy link
Author

@Lee-Lily-Lea 最小改动的方式只能是增加一个属性而不破坏现有的API,也许叫SelectOnRow

@capdiem 考虑兼容确实

另外 这个效果有几点注意

  1. 支持hover效果(可以不带 通过Hoverable=true实现也可以 但是不能不兼容)
  2. 开启以后 就不要选择到文字了 (user-select)
  3. 尽量是以多个无缝隙的row去点击 意思就是 不要出现在两行之间有空白 点了没反应

🚀你加油

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

No branches or pull requests

2 participants