Skip to content

[Frontend][A11y] Associate the post-revision Review lock reason with disabled decisions #2461

Description

@Chris0Jeky

Context

PR #2448 renders a live status while the post-revision truth refresh disables the Review decision rail. Independent accessibility review found the status is attached through aria-describedby to the non-focusable ReviewMain wrapper, while the four disabled decision controls only receive the older editLock description linkage. Refs #1964, #2215, and PR #2448.

Current impact

The live role=status announcement and focus restoration work, so the reviewer is not silently trapped. However, assistive technology inspecting the disabled Reject, Request edit, Defer, or Approve control does not get a direct relationship to the refresh-lock explanation.

Acceptance

  • Give ReviewDecisionRail an explicit refresh-lock reason or description id contract that reaches every disabled decision control.
  • Preserve the existing editing and saving lock descriptions and cancel behavior.
  • Avoid dangling aria-describedby references when no explanation is rendered.
  • Prove the association in an attached-DOM component test for all rendered decision controls and retain the browser focus-return proof.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Priority IIIValuable but unscheduled: residuals, tech-debt, performance, depth work.bugSomething isn't workingfrontendPrimary implementation impact in Vue/TypeScript UI and client runtime.uxDiscoverability, accessibility, interaction model, and user workflow quality.

    Projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions