Skip to content

fix(client): improve app list a11y - #1152

Merged
webfansplz merged 2 commits into
vuejs:mainfrom
shuaiyafei:fix/app-list-keyboard-accessibility
Oct 15, 2026
Merged

webfansplz merged 2 commits into
vuejs:mainfrom
shuaiyafei:fix/app-list-keyboard-accessibility

Conversation

@shuaiyafei

Copy link
Copy Markdown
Contributor

Summary

  • render frame and app entries as native buttons so they are keyboard focusable and activatable
  • expose the current frame and app selection with aria-pressed while preserving the existing styling and callbacks
  • add an integration test covering labels, selection state, focusability, and selection callbacks

Validation

  • pnpm vitest tests/integration/client/app-list.test.ts --run
  • pnpm format:check
  • pnpm typecheck
  • pnpm lint
  • pnpm build
  • pnpm test (61 files, 357 tests passed)
  • git diff --check

Relates to #885.

This is scoped to AppList.vue and does not overlap with #1146.

@netlify

netlify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for vue-devtools-docs ready!

Name Link
🔨 Latest commit 7dbd5ee
🔍 Latest deploy log https://app.netlify.com/projects/vue-devtools-docs/deploys/6ad032a66cc496a7a2f2f9c5
😎 Deploy Preview https://deploy-preview-1152--vue-devtools-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@webfansplz webfansplz changed the title fix(client): make app list keyboard accessible fix(client): improve app list a11y Oct 15, 2026
@pkg-pr-new

pkg-pr-new Bot commented Oct 15, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@vue/devtools-api@1152
npm i https://pkg.pr.new/@vue/devtools-kit@1152
npm i https://pkg.pr.new/vite-plugin-vue-devtools@1152

commit: 7dbd5ee

@webfansplz
webfansplz merged commit 09249b5 into vuejs:main Oct 15, 2026
7 checks passed
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.

2 participants