-
Notifications
You must be signed in to change notification settings - Fork 48
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
EmojiList should be keyboard accessible #104
Comments
@LukeGarrigan Thanks for the links, it looks doable. Basically it looks like Related to: #83. |
I started looking into it and while the mentioned change in the parent project improves accessibility behavior, it only works with special accessibility apps (like VoiceOver on mac). This is also confirmed in the comments to the pull request:
So this does not work with regular keyboard input (it won't magically enable arrow keys to select and enter emojis). Here is a good article on the topic that also shows how to build a full keyboard control using arrow keys: https://www.digitalocean.com/community/tutorials/vuejs-vue-a11y-autocomplete |
Thank you, so with your current changes the user will be able to tab through the emojis? |
@LukeGarrigan no, it won't work with regular keyboard input, only with special software like VoiceOver on mac (see an example here - https://a11y-101.com/development/landmarks, there is a video showing how it works). I started working on actual keyboard control (like in Slack), but it is more complex as I acually have to handle key events and track the active category and emoji. |
[#104] Accessibility improvements and keyboard controls
I've just published version 10.0.0 with keyboard controls and accessibility improvements (use Keyboard controls are activated when the cursor is in the search field - emoji can be selected with arrow keys and confirmed with Enter (the |
Awesome, thank you @serebrov |
When focusing on the emoji-picker the user should be able to use arrow-keys or tab to navigate through the list of emojis.
A similar change was made to the main parent of this project (emoji-mart)
Issue: missive#218
PR: missive#284
It would be great if a similar change would be made in this project!
The text was updated successfully, but these errors were encountered: