Skip to content

Fix 1876 a11y#2018

Merged
corinagum merged 8 commits intomicrosoft:masterfrom
corinagum:fix-1876-a11y
May 28, 2019
Merged

Fix 1876 a11y#2018
corinagum merged 8 commits intomicrosoft:masterfrom
corinagum:fix-1876-a11y

Conversation

@corinagum
Copy link
Copy Markdown
Contributor

@corinagum corinagum commented May 21, 2019

Fixes #1876

Changelog Entry

Description

This was an extended dive into a11y so that the Narrator, NVDA, and VoiceOver will only read changes in the transcript once

Specific Changes

  • Explicitly designate the aria-atomic and aria-relevant attributes
  • Remove nested live attributes
  • Remove timestamp from the "User sent __" and "Bot sent __" aria-label strings
  • Change aria-label of the component to be an absolute timestamp for the "xMinutesAgo" feature, but remain as "Sending" before send has completed. This will presumably prevent the screen reader from repeatedly updating all es in the transcript.
  • A chart confirming which browsers and what screen reader we expect Web Chat to work on.

Copy link
Copy Markdown
Contributor

@compulim compulim left a comment

Choose a reason for hiding this comment

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

Looks good to me, just nitpicking.

Comment thread CHANGELOG.md Outdated
Comment thread packages/component/src/Activity/Bubble.js
Comment thread packages/component/src/Activity/Timestamp.js
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.

Screen readers narrate all the messages from the beginning for every new message

2 participants