-
Notifications
You must be signed in to change notification settings - Fork 192
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
chore(switch): update swipe interaction docs #1096
base: main
Are you sure you want to change the base?
Conversation
After resolving openui#1045, this pull request removes the question and defines a swipe interaction with reference to the resolution.
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.
Left a minor request but approving with changes requested. Please close out the related issue upon merge.
From a user standpoint, the following interactions should be supported | ||
|
||
- Click/tap: toggles the switch | ||
- <kbd>Spacebar</kbd>: toggles the switch which has keyboard focus | ||
- Swipe: Switch should support gesture support akin to the range control ([#1045](https://github.com/openui/open-ui/issues/1045)) |
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.
Can you please link to the MDN documentation of the range control?
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.
Provided link to MDN per @gregwhitworth request.
Co-authored-by: Tantek Çelik <[email protected]>
After resolving #1045, this pull request removes the question and defines a swipe interaction with reference to the resolution.