Skip to content

Fix usability on accessibility#3076

Merged
compulim merged 8 commits intomicrosoft:masterfrom
compulim:exp-a11y-humana
Apr 3, 2020
Merged

Fix usability on accessibility#3076
compulim merged 8 commits intomicrosoft:masterfrom
compulim:exp-a11y-humana

Conversation

@compulim
Copy link
Copy Markdown
Contributor

@compulim compulim commented Apr 3, 2020

Fix #3075.

Changelog Entry

Fixed

  • Fixes #3075. Fix usability issues around accessibility, by @compulim in PR #3076
    • Fix timestamp should not be narrated more than once.
    • Associate the activity text with its attachments, by adding a role="region" to the activity DOM element.

Description

(See changelog above)

This may be a temporary fix until we found a better DOM structure.

Specific Changes

  • Update <StackLayout> to narrate as aria-label on the activity root DOM element
  • Update <CarouselFilmstrip> to match <StackLayout> on screen reader text
  • Update aria-relevant for <BasicTranscript> to only narrate additions of node
  • Remove <Timestamp> from narration, because it will be narrated by the activity

Test

  1. Run Windows Narrator (Windows + CTRL + ENTER)
  2. Say "hello" to the bot
    • Expect: The narration should be minimal and not repetitive
  3. Say "card inputs breakfast"
  4. Use mouse to focus on any box of "inputs card"
  5. Use mouse to focus on the button on "breakfast card"
    • Expect: The narration should mention it's on breakfast card
  6. Use mouse to focus on the dropbox of "breakfast card"
    • Expect: The narration should not mention it's on breakfast card again
  7. Use mouse to focus on any input on "card inputs"
    • Expect: The narration should mention it's on inputs card

  • Testing Added
    • No tests added because accessibility cannot be automated

Comment thread packages/component/src/Activity/StackedLayout.js Outdated
Copy link
Copy Markdown
Contributor

@corinagum corinagum left a comment

Choose a reason for hiding this comment

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

Approved; See nit

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Accessibility: screen reader repeat timestamp multiple times

2 participants