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 extension keyboard accessible #139

Open
pjeweb opened this issue Oct 8, 2023 · 9 comments
Open

Make extension keyboard accessible #139

pjeweb opened this issue Oct 8, 2023 · 9 comments
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Issues great for Hacktoberfest contributors help wanted Extra attention is needed

Comments

@pjeweb
Copy link
Member

pjeweb commented Oct 8, 2023

Make the extension work with keyboard input:

  • make keyboard focus visible
  • allow closing modals with esc
  • prevent tabbing outside of modals or into hidden elements
  • to not move or loose focus when interacting
  • buttons and other elements should follow common input patterns (space/enter, cursor keys, escape etc.)
  • probably forgot some points otomh
@pjeweb pjeweb added the enhancement New feature or request label Oct 8, 2023
@prSquirrel
Copy link

I think it would be simpler to use some kind of ARIA-aware library for that. I’ve played with react-aria hooks for a bit and it works pretty well. The focus is managed correctly and directional arrows are supported too. The only inconvenience is many hooks depend on their own react-stately state management lib, but that can be hidden inside component.

Could you please assign this issue to me?

@pjeweb
Copy link
Member Author

pjeweb commented Dec 27, 2023

That makes sense to me. I think the main goal is to keep the visual appearance and features the same, but I assume that was obvious.

I'd want @MattIPv4 input on the use of react-aria. I'm using that for the website too, and I do like it quite a lot, so it would make sense to use it for the extension too imho.

@MattIPv4
Copy link
Member

Where we can, making sure we're using semantic elements and native events would be best, but if there are cases where we can't do that easily, react-aria seems fine.

@MattIPv4

This comment was marked as outdated.

@prSquirrel

This comment was marked as outdated.

@MattIPv4 MattIPv4 added help wanted Extra attention is needed good first issue Good for newcomers hacktoberfest Issues great for Hacktoberfest contributors labels Sep 22, 2024
@mutant138

This comment was marked as outdated.

@MattIPv4

This comment was marked as outdated.

@MattIPv4

This comment was marked as outdated.

@mutant138

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Issues great for Hacktoberfest contributors help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants