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

feat: add keyboard navigation to svelte inspector and improve a11y #438

Merged
merged 7 commits into from Sep 17, 2022

Conversation

dominikg
Copy link
Member

@dominikg dominikg commented Sep 15, 2022

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.

Copy link
Member

@geoffrich geoffrich left a 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
@dominikg
Copy link
Member Author

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?

@bluwy
Copy link
Member

bluwy commented Sep 17, 2022

I think it'd be fine if we stick as a patch 👍

@dominikg dominikg changed the title feat: add next/prev keyboard navigation to svelte inspector and improve a11y feat: add keyboard navigation to svelte inspector and improve a11y Sep 17, 2022
@dominikg dominikg merged commit 7796a8e into main Sep 17, 2022
@dominikg dominikg deleted the feat/inspector-a11y branch September 17, 2022 16:00
@github-actions github-actions bot mentioned this pull request Sep 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants