Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Screen reader said the avatar initials without context #1499

Closed
compulim opened this issue Dec 14, 2018 · 1 comment
Closed

Screen reader said the avatar initials without context #1499

compulim opened this issue Dec 14, 2018 · 1 comment
Assignees
@corinagum
Labels
area-accessibility front-burner p1 Painful if we don't fix, won't block releasing size-s 1 days or less
Milestone
v4.3

Comments

@compulim
Copy link
Contributor

compulim commented Dec 14, 2018 •
edited

Background

When screen reader goes thru avatar initials, it will say "WC". The user may not aware WC is initials of the bot or the user.

Note to fixer

  • Use aria-label so the screen reader will say something like "Avatar of bot: WC"
  • Use role to image?
@compulim compulim added area-accessibility 4.3 p1 Painful if we don't fix, won't block releasing labels Dec 14, 2018
@compulim
Copy link
Contributor Author

@corinagum can you help on this one?

@compulim compulim mentioned this issue Dec 14, 2018
Merged
@compulim compulim added this to the v4.3 milestone Dec 18, 2018
corinagum pushed a commit to corinagum/BotFramework-WebChat that referenced this issue Dec 21, 2018
microsoft#1499 Fix screen reader handling of name, activity, & timestamp
5ae941c
@corinagum corinagum mentioned this issue Dec 21, 2018
Merged
@corinagum corinagum added size-s 1 days or less and removed T-Shirt: XS labels Dec 21, 2018
corinagum pushed a commit to corinagum/BotFramework-WebChat that referenced this issue Dec 21, 2018
microsoft#1499 Fix screen reader handling of name, activity, & timestamp
71f6758
corinagum pushed a commit that referenced this issue Dec 21, 2018
#1499 Fix screen reader handling of name, activity, and timestamp (#1528
81b9e0a 
)

* #1499 Fix screen reader handling of name, activity, & timestamp

* Update CHANGELOG.md

* PR Fixes
@corinagum corinagum removed this from P1 in 4.3 Prioritization Dec 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@corinagum corinagum

Labels
area-accessibility front-burner p1 Painful if we don't fix, won't block releasing size-s 1 days or less
Projects
None yet
Milestone
v4.3
Development

No branches or pull requests
2 participants
@compulim @corinagum