Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 7, 2026

Agent session aria labels announced session type before label, violating accessibility best practices of prioritizing critical information.

Changes

  • AgentSessionsAccessibilityProvider: Reordered localization parameters in getAriaLabel() to place element.label before element.providerLabel

    • Before: "Background session Explain this screenshot (Completed), created..."
    • After: "Explain this screenshot Background session (Completed), created..."
  • Test coverage: Added agentSessionsAccessibilityProvider.test.ts with assertions verifying label precedence in aria output

Original prompt

This section details on the original issue you should resolve

<issue_title>Aria label of session entries should have type of session at the end</issue_title>
<issue_description>Right now the aria label seems to be TYPE LABEL. Instead it should be LABEL TYPE.
The practice we should follow is to always first announce the most important information to the user, and that is the label of the chat session

Image

@joshspicer @meganrogge </issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update aria labels for session entries Reorder aria label for agent sessions to announce label before type Jan 7, 2026
Copilot AI requested a review from bpasero January 7, 2026 16:58
@meganrogge meganrogge added this to the January 2026 milestone Jan 7, 2026
@bpasero bpasero closed this Jan 8, 2026
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.

Aria label of session entries should have type of session at the end

3 participants