-
-
Notifications
You must be signed in to change notification settings - Fork 110
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
feat: add keyboard navigation to svelte inspector and improve a11y #438
Conversation
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.
One small comment. The announcer stuff seems like it should work, but you'd probably want to test with VoiceOver or something to be sure.
…n navigated element, clamp y pos of bubble to viewport
i also cleaned up the changeset files, hoping it creates a cleaner changelog for 1.1.0. given inspector is still experimental and is going to move into a separate package at a later stage, should we use patch instead? |
I think it'd be fine if we stick as a patch 👍 |
this PR follows up on #436 to complete keyboard only navigation and a11y improvements
There are two new options for inspector. navKeys (default to arrow keys) and openKey (defaults to Enter).
Read more here https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/config.md#inspector
This allows you to operate svelte-inspector by keyboard only.
In addition the currently selected dom node's file position is announced via aria-live for screen-readers.