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

Improve keyboard accessibility #13

Open
jeffpaul opened this issue Jul 19, 2019 · 5 comments
Open

Improve keyboard accessibility #13

jeffpaul opened this issue Jul 19, 2019 · 5 comments
Labels
help wanted Extra attention is needed type:accessibility An issue with accessibility. type:bug Something isn't working.

Comments

@jeffpaul
Copy link
Member

No description provided.

@jeffpaul jeffpaul added the type:bug Something isn't working. label Jul 19, 2019
@jeffpaul jeffpaul added this to the 1.0.0 milestone Jul 19, 2019
@adamsilverstein
Copy link

Opened matching issue in the upstream repository:

Dayjo/react-character-map#7

Opened PR addressing issue in upstream repo: Dayjo/react-character-map#8

Would appreciate a more expert accessibility review :)

@helen
Copy link
Contributor

helen commented Aug 31, 2019

Tabbing around works decently now, the things I'm not sure about that perhaps @samikeijonen would know are:

  • Is it an issue that the categories/tabs disappear while filtering?
  • Would it be helpful to have escape go back to the categories/tabs if you are currently focused on a character item? So that if it's a long list you don't have to tab forever to get back. It may not be, because you can escape to close and then reopen the modal, but I'm curious about expectations for what escape does.

@samikeijonen
Copy link
Contributor

I don't have the plugin open in my dev environment at the moment but I'm going to reply now so that I don't forget:)

Is it an issue that the categories/tabs disappear while filtering?

I don't see that big issue. But let's say categories/tabs doesn't disappear while filtering. What would happen when my filter finds for example 10 characters?

  • Would categories/tabs filter them even more depending where those 10 characters belong?
  • What if out of that 10 characters all are in one category. Should all categories/tabs still be visible? I'd say no since they don't do anything at this point.

Would it be helpful to have escape go back to the categories/tabs if you are currently focused on a character item?

My first reaction is that escape should close the modal. That's what's happening all over the block editor UI.

@mrwweb
Copy link

mrwweb commented Sep 24, 2019

A few notes:

  1. TABing

Tabbing around works decently now

With Firefox 70 and Windows 10, that's not the case for me. I can open the modal with a keyboard, but then can only TAB between the modal container and the filter input. I can search, but I can't find a way to actually select and insert a character.

  1. Using the arrow keys (the next thing I tried) seems to move my focus back into the document (apparently from the title or first block?).

  2. For one example of my dream keyboard functions, I love the Windows 10 emoji inserter (WIN + ; or WIN + .).

animated gif showing windows emoji keyboard functionality

@adamsilverstein
Copy link

Thanks for the feedback and windows emoji inserter screencast @mrwweb - I can confirm we noticed the same on mac: Firefox keyboard tabbing does not work as expected. In addition, when I had the focus set to go into the search box on opening, Firefox failed to open the modal at all (see #41).

I worked on this a bit and couldn't quite figure out what needs to change. Maybe I can get another review from @samikeijonen of the generated HTML - something needs to change there either in the plugin or upstream.

@jeffpaul jeffpaul added the help wanted Extra attention is needed label Oct 14, 2021
@cldhmmr cldhmmr self-assigned this Oct 19, 2021
@dkotter dkotter modified the milestones: 1.0.5, 1.1.0 Jun 24, 2022
@jeffpaul jeffpaul modified the milestones: 1.1.0, Future Release Jun 28, 2022
@jeffpaul jeffpaul added the type:accessibility An issue with accessibility. label Jun 28, 2022
@mphillips mphillips assigned mphillips and unassigned mphillips Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed type:accessibility An issue with accessibility. type:bug Something isn't working.
Projects
None yet
Development

No branches or pull requests

8 participants