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.

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

Fix aria-label of Sendbox #2423

Merged
merged 7 commits into from
Sep 24, 2019
Merged

Conversation

corinagum
Copy link
Contributor

@corinagum corinagum commented Sep 23, 2019

Fixes microsoft/BotFramework-Emulator#1823

Changelog Entry

Description

This is an accessibility fix for the sendbox in Web Chat, where the narrator was reading the placeholder twice. Now it will say, "Sendbox edit 'Type your message'" instead of "Type your message edit type your message"

@coveralls
Copy link

coveralls commented Sep 23, 2019

Coverage Status

Coverage increased (+0.03%) to 65.338% when pulling 22ccaa6 on corinagum:emulator1823 into e54d213 on microsoft:master.

CHANGELOG.md Outdated
@@ -87,6 +87,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
- Fix [#2365](https://github.com/microsoft/BotFramework-WebChat/issues/2365). Fix Adaptive Card `pushButton` appearance on Chrome, by [@corinagum](https://github.com/corinagum) in PR [#2382](https://github.com/microsoft/BotFramework-WebChat/pull/2382)
- Fix [#2379](https://github.com/microsoft/BotFramework-WebChat/issues/2379). Speech synthesis can be configured off by passing `null`, by [@compulim](https://github.com/compulim) in PR [#2408](https://github.com/microsoft/BotFramework-WebChat/pull/2408)
- Fix [#2418](https://github.com/microsoft/BotFramework-WebChat/issues/2418). Connectivity status should not waste-render every 400 ms, by [@compulim](https://github.com/compulim) in PR [#2419](https://github.com/microsoft/BotFramework-WebChat/pull/2419)
- Fix [Emulator1823](https://github.com/microsoft/BotFramework-Emulator/issues/1823). Fix Sendbox "Type your message" being read twice by AT, by [@corinagum](https://github.com/corinagum) in PR [#2423](https://github.com/microsoft/BotFramework-WebChat/pull/2423)
Copy link
Contributor

Choose a reason for hiding this comment

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

Emulator1823? 🤣

CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

@tdurnford tdurnford left a comment

Choose a reason for hiding this comment

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

LGTM

Co-Authored-By: TJ Durnford <tjdford@gmail.com>
CHANGELOG.md Outdated Show resolved Hide resolved
Co-Authored-By: TJ Durnford <tjdford@gmail.com>
@corinagum corinagum merged commit a312e1c into microsoft:master Sep 24, 2019
@compulim compulim mentioned this pull request Oct 4, 2019
13 tasks
compulim pushed a commit to compulim/BotFramework-WebChat that referenced this pull request Oct 5, 2019
* Fix aria-label of Sendbox

* Edit CHANGELOG.md

* Add localization

* Update CHANGELOG.md

Co-Authored-By: TJ Durnford <tjdford@gmail.com>

* Update CHANGELOG.md

Co-Authored-By: TJ Durnford <tjdford@gmail.com>
@compulim compulim mentioned this pull request Oct 25, 2019
55 tasks
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.

[Usability][Screen reader-Welcome] Nvda is reading twice for check boxes information
4 participants