-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
[color-scale] New selectable
prop
#179
Comments
This would make color swatches focusable and selectable, either by clicking or with the keyboard (and would give them appropriate ARIA semantics). When a swatch is selected, it would fire a suitable event. My use case was that I want an editable color scale that the user can add colors to, but also select the colors and have them reflected in a |
From a "classic" UI perspective, the swatches would be like radio buttons (single select) or checkboxes (multi-select, if you allow it). You might also need a |
All good points. The HTML way to specify multiple selection is a As I said in another issue, given how much extra code it took to make |
Are you sure you want to support multiple selection? Not many use cases for picking multiple colors at once, and here you have the additional limit that they're all in the same scale. I don't know how much multi-select might complicate this implementation. Multi-select adds the functionality of deselecting items. That lets the user leave nothing selected, which single-select effectively prohibits if you have a default selection, or after the user has selected an item. Scrolling means the user can't see everything that's selected at once unless there's a consolidated list of selected items alongside. Some UIs force you to use the shift and ctrl keys to multi-select, others don't. Et cetera. Multi-select tends to be a pain for everyone, programmer and user. But if you need it, you need it.
I keep mistakenly thinking it's |
Perhaps the interactive versions of things could be |
I think it makes sense. As an alternative, |
|
I like the conciseness of "live", but I think it's a bit too vague to communicate purpose here.
I think that's a feature, not a bug.
There is — just not directly by the user. |
Proposed by @LeaVerou. Lea, could you please elaborate on how this prop should be used and in what use cases?
The text was updated successfully, but these errors were encountered: