Skip to content

fix(focus-preview): name the card Focus preview, show it on toggle, wait for scans (0.15.6) - #28

Merged
DuncanFaulkner merged 4 commits into
mainfrom
fix/focus-preview-naming
Oct 2, 2026
Merged

DuncanFaulkner merged 4 commits into
mainfrom
fix/focus-preview-naming

Conversation

@DuncanFaulkner

Copy link
Copy Markdown
Contributor
  • The Focus preview card is now headed "Focus preview — computed approximation", matching the pill-menu setting (it said "Accessibility-tree preview").
  • Turning Focus preview on describes the already-focused control right away. When toggled from the pill menu, it describes the page control focused before. Turning the tool on does the same once its first scan finishes.
  • Fix: a control focused mid-scan showed "(no accessible name)", because axe.setup throws while axe.run owns the tree. The preview now waits for scans to go idle (capped at 3), and only the latest focus is described. describeElement no longer tears down a scan's tree after a failed setup, which silently dropped that scan's findings.
  • Tests for the menu toggle, the shortcut toggle, focus mid-scan, and switching off while waiting.

🤖 Generated with Claude Code

DuncanFaulkner and others added 4 commits October 2, 2026 22:06
…ait for scans

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… scan's axe tree; tests

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@DuncanFaulkner
DuncanFaulkner merged commit a530f77 into main Oct 2, 2026
1 check passed
@DuncanFaulkner
DuncanFaulkner deleted the fix/focus-preview-naming branch October 2, 2026 21:25
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.

1 participant