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

Make emojis selectable by tab #2552

Merged
merged 2 commits into from May 5, 2022
Merged

Make emojis selectable by tab #2552

merged 2 commits into from May 5, 2022

Conversation

skjnldsv
Copy link
Contributor

@skjnldsv skjnldsv commented Mar 16, 2022

image

  • on open : Search is in focus

  • if search is in focus, emojis are selectable by arrow keys

  • Categories and emojis are selectable by tab

Is this the behavior we want to see?

@skjnldsv skjnldsv added enhancement New feature or request 2. developing Work in progress accessibility Making sure we design for the widest range of people possible, including those who have disabilities feature: emoji picker Related to the emoji picker component labels Mar 16, 2022
@vanpertsch vanpertsch changed the title Make categories selectable by tab Make selectable by tab Mar 17, 2022
@vanpertsch vanpertsch marked this pull request as ready for review March 17, 2022 09:37
@nimishavijay
Copy link

on open : Search is in focus
if search is in focus, emojis are selectable by arrow keys

Looks good! :)

Categories and emojis are selectable by tab

Not sure about this, I think arrow keys can be used to select emojis and categories also. Tab can used to change focus between categories, search, and emojis. Within that you navigate using arrow keys.

Also a couple more points from me:

  • Esc should close the picker
  • If you have selected the last emoji in the list, pressing the right arrow should cycle back to the first one again. Currently the right arrow does nothing if you are on the last emoji. Similarly for left arrow and first emoji as well.
  • I also think that if the emoji picker is open, tab should not make focus leave the picker, it should only cycle through categories, search and emojis. Currently if you are focused on the last emoji and you press tab, the focus jumps to the browser tab which is a bit jarring. What do you think?

cc @jancborchardt if there is anything else to add

@skjnldsv
Copy link
Contributor Author

  • If you have selected the last emoji in the list, pressing the right arrow should cycle back to the first one again. Currently the right arrow does nothing if you are on the last emoji. Similarly for left arrow and first emoji as well.
  • I also think that if the emoji picker is open, tab should not make focus leave the picker, it should only cycle through categories, search and emojis. Currently if you are focused on the last emoji and you press tab, the focus jumps to the browser tab which is a bit jarring. What do you think?

Not sure those two are doable

@jancborchardt
Copy link
Contributor

Only a detail from my side: Would look nicer to have the focus box be a circle instead of a square with rounded corners? :)

@vanpertsch vanpertsch force-pushed the a11y-emoji-picker branch 2 times, most recently from 048c2d4 to 23d20c4 Compare March 23, 2022 09:36
@vanpertsch

This comment was marked as resolved.

@skjnldsv skjnldsv changed the title Make selectable by tab Make emojis selectable by tab Mar 23, 2022
@skjnldsv
Copy link
Contributor Author

/rebase

@vanpertsch vanpertsch force-pushed the a11y-emoji-picker branch 4 times, most recently from 349650b to 79528e5 Compare April 20, 2022 13:22
@vanpertsch
Copy link
Contributor

vanpertsch commented Apr 20, 2022

@nimishavijay & @jancborchardt can you check again if the picker looks good and works as expected like it is now?

  • If you have selected the last emoji in the list, pressing the right arrow should cycle back to the first one again. Currently the right arrow does nothing if you are on the last emoji. Similarly for left arrow and first emoji as well.
  • is not possible (atm)
  • I also think that if the emoji picker is open, tab should not make focus leave the picker, it should only cycle through categories, search and emojis. Currently if you are focused on the last emoji and you press tab, the focus jumps to the browser tab which is a bit jarring.
  • focus stays inside

Search is in focus on open
Screenshot 2022-04-20 at 15 24 36|Screenshot 2022-04-20 at 15 46 25

Categories selectable by tab
Screenshot 2022-04-20 at 15 24 25|Screenshot 2022-04-20 at 15 48 37

emojis selectable by tab and arrow-keys
Screenshot 2022-04-20 at 15 24 47|Screenshot 2022-04-20 at 15 47 04

@skjnldsv
Copy link
Contributor Author

Approval from my side, you can self-approve on my behalf @vanpertsch
:)

@vanpertsch vanpertsch self-requested a review April 21, 2022 14:00
src/components/EmojiPicker/EmojiPicker.vue Show resolved Hide resolved
src/components/EmojiPicker/EmojiPicker.vue Outdated Show resolved Hide resolved
src/components/EmojiPicker/EmojiPicker.vue Outdated Show resolved Hide resolved
src/components/EmojiPicker/EmojiPicker.vue Show resolved Hide resolved
src/components/EmojiPicker/EmojiPicker.vue Show resolved Hide resolved
src/components/EmojiPicker/EmojiPicker.vue Outdated Show resolved Hide resolved
Signed-off-by: Vanessa Pertsch <vanessa.pertsch@nextcloud.com>
Signed-off-by: Vanessa Pertsch <vanessa.pertsch@posteo.de>
Copy link

@nimishavijay nimishavijay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 :)

Signed-off-by: Vanessa Pertsch <vanessa.pertsch@posteo.de>
Copy link
Contributor

@artonge artonge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@vanpertsch vanpertsch merged commit 621d795 into master May 5, 2022
@vanpertsch vanpertsch deleted the a11y-emoji-picker branch May 5, 2022 08:20
@juliushaertl juliushaertl mentioned this pull request Aug 2, 2022
@juliushaertl juliushaertl added this to the 5.4.0 milestone Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress accessibility Making sure we design for the widest range of people possible, including those who have disabilities enhancement New feature or request feature: emoji picker Related to the emoji picker component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants