Skip to content

docs(wiki): say the A2 message is written to the live region, not that it is spoken - #60

Merged
hyperpolymath merged 1 commit into
mainfrom
docs/a2-live-region-wording
Oct 6, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
docs/a2-live-region-wording

Conversation

@hyperpolymath

Copy link
Copy Markdown
Contributor

Summary

This PR fixes one sentence in docs/wiki/users/Feature-Status.adoc, a follow-up to #58 raised by CodeRabbit on that PR (comment 4198167436).

  • Before: "A change that switches other blocks is announced once to screen readers."
  • After: the change "writes one message to the block's role="status" live region (RELEASE-CRITERIA A2); that a screen reader speaks it is not yet verified (A1)."

The A2 tests read the region's text under happy-dom. No screen-reader run has confirmed speech, so the page no longer claims it.

Closes # — no issue; this is review follow-up to #58.

Type of change

  • 🐛 Bug fix: no code change.
  • ✨ New feature: none.
  • 💥 Breaking change: none.
  • 🕳️ Soundness fix: not a checker fix, though it removes an overclaim.
  • 📖 Documentation
  • 🧹 Refactor / tech debt: none.
  • ⚡ Performance: none.
  • 🔧 Build / CI / tooling: none.

📌 New pins

  • Head SHA: 59dc16e
  • New or changed pins: none.

How has this been verified?

Checklist

  • My commits are signed: git log -1 --format=%G? → G.
  • I ran the project's own checks locally (wiki health), and they pass.
  • New files carry the correct SPDX identifier: no new files.
  • Docs are updated, and no claim overstates the code. Removing an overclaim is the point of this PR.
  • No soundness hole introduced: docs only.

Notes for reviewers

🤖 Generated with Claude Code

https://claude.ai/code/session_019tmsGh5n4GwJZYq9Z5mtxH

…t it is spoken

Feature-Status claimed the group-sync change is "announced once to screen
readers". The tests read the status region's text under happy-dom; no
screen-reader run has confirmed speech (A1). Raised by CodeRabbit on #58.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019tmsGh5n4GwJZYq9Z5mtxH
@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 43 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 98972a16-f096-4b47-8479-57bc7c7286d7
📥 Commits

Reviewing files that changed from the base of the PR and between fa91850 and 59dc16e.

📒 Files selected for processing (1)
  • docs/wiki/users/Feature-Status.adoc
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hyperpolymath
hyperpolymath merged commit bc3be0d into main Oct 6, 2026
14 checks passed
@hyperpolymath
hyperpolymath deleted the docs/a2-live-region-wording branch October 6, 2026 17:26
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