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

Accessibility: Narrator reads markdown that is passed through the localize method #2001

Closed
corinagum opened this issue May 17, 2019 · 0 comments · Fixed by #2096
Closed

Accessibility: Narrator reads markdown that is passed through the localize method #2001

corinagum opened this issue May 17, 2019 · 0 comments · Fixed by #2096
Assignees
@corinagum
Labels
area-accessibility bug Indicates an unexpected problem or an unintended behavior. front-burner p1 Painful if we don't fix, won't block releasing

Comments

@corinagum
Copy link
Contributor

Describe the bug

To Reproduce

Steps to reproduce the behavior:

  1. Turn on Narrator
  2. Go to full-feature Mock Bot
  3. Type 'timestamp' and send
  4. Note that Narrator says message "First message at time t = 0." as: "First message at time underscore t underscore equals 0"

Expected behavior

Markdown should be stripped from text before being passed to localize

Additional context
use the remark plugin called 'strip markdown'

[Bug]
@corinagum corinagum added bug Indicates an unexpected problem or an unintended behavior. area-accessibility Approved front-burner labels May 17, 2019
@corinagum corinagum mentioned this issue May 21, 2019
Closed
@corinagum corinagum added 4.5 p2 Nice to have and removed Triage-E labels May 23, 2019
@corinagum corinagum added p1 Painful if we don't fix, won't block releasing and removed p2 Nice to have labels Jun 10, 2019
@corinagum corinagum mentioned this issue Jun 19, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@corinagum corinagum

Labels
area-accessibility bug Indicates an unexpected problem or an unintended behavior. front-burner p1 Painful if we don't fix, won't block releasing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove markdown from spoken text for screen reader corinagum/BotFramework-WebChat
1 participant
@corinagum