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

Carousel Layout text activities are repeated multiple times by AT due to nesting <Bubble> #2277

Closed
corinagum opened this issue Aug 6, 2019 · 0 comments · Fixed by #2613
Closed

Carousel Layout text activities are repeated multiple times by AT due to nesting <Bubble> #2277

corinagum opened this issue Aug 6, 2019 · 0 comments · Fixed by #2613
Assignees
@corinagum
Labels
area-accessibility backlog Out of scope for the current iteration but it will be evaluated in a future release. bug Indicates an unexpected problem or an unintended behavior. front-burner

Comments

@corinagum
Copy link
Contributor

corinagum commented Aug 6, 2019 •
edited
Loading

Version 4.5.1

Describe the bug

To Reproduce

Steps to reproduce the behavior:

  1. Open Mock Bot
  2. Turn on Narrator (Edge) or NVDA (Chrome)
  3. Type 'layout carousel' into Mock Bot
  4. Note that the text activities (non-Markdown) are read multiple times

Expected behavior

Text activities in carousels should only be read once. This is caused by the nesting of in <CarouselLayout>.

Additional context

Uncovered from issue #2273

[Bug]

Note to fixer:
also test Mock Bot command layout
@corinagum corinagum added bug Indicates an unexpected problem or an unintended behavior. Triage-E and removed Customer labels Aug 6, 2019
@corinagum corinagum mentioned this issue Aug 6, 2019
Merged
1 task
@corinagum corinagum self-assigned this Aug 6, 2019
@corinagum corinagum changed the title Carousel Layout text activities are repeated multiple times due to nesting <Bubble> Carousel Layout text activities are repeated multiple times by AT due to nesting <Bubble> Aug 8, 2019
@corinagum corinagum added backlog Out of scope for the current iteration but it will be evaluated in a future release. front-burner and removed Triage-E labels Nov 7, 2019
@corinagum corinagum mentioned this issue Nov 20, 2019
Merged
1 task
@compulim compulim mentioned this issue Dec 10, 2019
Closed
73 tasks
@corinagum corinagum mentioned this issue Dec 13, 2019
Closed
73 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@corinagum corinagum

Labels
area-accessibility backlog Out of scope for the current iteration but it will be evaluated in a future release. bug Indicates an unexpected problem or an unintended behavior. front-burner
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make Suggested Actions accessible again! corinagum/BotFramework-WebChat
1 participant
@corinagum