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

[Accessibility][Headings and Labels - Flow Chart] : The name of the flow chart should be defined under heading tag. #2028

Closed
ashish315 opened this issue Feb 20, 2020 · 3 comments · Fixed by #2128

Comments

@ashish315
Copy link

User Experience
Users who rely on AT will face difficulty and easily confused if the headings are not defined correctly.

Test Environment
Browser: Microsoft Edge Canary Version: 82.0.425.0 (Official build)
OS build: 2004 (19564.1005)
Screen Reader: Narrator
URL: Bot Framework Composer
Prerequisite Here

Repro Steps:

  1. Open the above mentioned URL in EDGE.
  2. Select 'Design Flow' from left navigation.
  3. Now, turn on the narrator and navigate with 'H' i.e. shortcut for Headings.
  4. Observe the issue in the flow chart area.

Actual:
The heading of the flow chart i.e. 'Dialog started' is not defined under heading tags.

Expected:
The name of the flow chart should be defined under heading tag. So, that the user will able to navigate quickly through the heading shortcut.

MAS Impacted: MAS2.4.6

Attachment:
Heading is not defined for the flow chart

@ashish315
Copy link
Author

@corinagum As checked, now it is defined in heading {specifically in h4} but the heading structure is not logically correct as no h1, h2 or h3 is defined on the page.

@ashish315 ashish315 removed the Fixed label Mar 12, 2020
@corinagum
Copy link
Contributor

@ashish315 thank you verifying. I have moved this back to the To Do column.

@ashish315
Copy link
Author

As checked and verified on below environment, the issue is fixed now. So, we are closing the issue.

Test Environment
Browser: Microsoft Edge Dev {Version 83.0.467.0 (Official build) dev (64-bit)}
OS build: 2004 (19588.1000)
Screen Reader: Narrator
URL: Bot Framework Composer
Prerequisite Here

@ashish315 ashish315 removed the Fixed label Apr 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment