Skip to content

a11y: add semantic roles, aria-live, and keyboard navigation to main containers - #1181

Open
Aryan-Verma-999 wants to merge 1 commit into
RocketChat:developfrom
Aryan-Verma-999:a11y/add-semantic-roles-keyboard-nav
Open

a11y: add semantic roles, aria-live, and keyboard navigation to main containers#1181
Aryan-Verma-999 wants to merge 1 commit into
RocketChat:developfrom
Aryan-Verma-999:a11y/add-semantic-roles-keyboard-nav

Conversation

@Aryan-Verma-999

@Aryan-Verma-999 Aryan-Verma-999 commented Mar 3, 2026

Copy link
Copy Markdown
Contributor
  • Message list: role="log" + aria-live="polite" so screen readers announce new messages
  • Channel name: role="button" + keyboard nav (Enter/Space) so it's accessible without a mouse
  • Member list: role="list"/role="listitem" structure + keyboard nav on each member item
    WCAG: Addresses 2.1.1 (Keyboard), 4.1.2 (Name, Role, Value), 4.1.3 (Status Messages)
    PR Test Details
    Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-1181 after approval. Contributors are requested to replace <pr_number> with the actual PR number.

@Aryan-Verma-999
Aryan-Verma-999 force-pushed the a11y/add-semantic-roles-keyboard-nav branch from 9539e89 to 1f96fa6 Compare August 27, 2026 13:59
@Spiral-Memory

Copy link
Copy Markdown
Collaborator

@Aryan-Verma-999 Is it still a valid PR ? Do you need review on this ?

@Aryan-Verma-999

Aryan-Verma-999 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor Author

It's an old pr, but yeah, it's still valid

@Spiral-Memory Spiral-Memory left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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