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][Screen reader - Property Pane_Ask a question (Bot Asks)]: Incorrect role is announced for ? as button in property pane. #2115

Closed
ashish315 opened this issue Feb 27, 2020 · 9 comments

Comments

@ashish315
Copy link

ashish315 commented Feb 27, 2020

User Experience
User who rely on screen reader will get confused if incorrect role is announced by the screen reader.

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 and turn on narrator.
  2. Navigate to already created dialog tree item and activate it. Such as- Greetings, Actions, EndDialog etc.
  3. Click on "+" and select 'Ask a question'. Then select any of the available options except Oauth login.
  4. Navigate to property pane appeared at right side of the screen.
  5. Now, navigate to '?' just beside of 'Prompt.
  6. Observe the issue.

Actual:
On getting the focus, narrator announces role as button along with the descriptive information.

Expected:
As it has no functionality, the narrator shouldn't announce the control as button.

Note:
Same issue is observed for all the Property Pane which have "?".

MAS Reference: MAS4.2.1

Attachment:
MAS4.1.2_incorrect role is defined.zip
MAS4 1 2_Incorrect role is defined for question mark button

@ashish315
Copy link
Author

@cwhitten @beyackle As checked on below environment the issue is still repro.

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

@ashish315
Copy link
Author

@cwhitten Stil narrator announces as '?' as button.

@cwhitten
Copy link
Member

cwhitten commented Apr 8, 2020

@ashish315 what branch are you running on the Composer project to test these fixes?

@ashish315
Copy link
Author

@cwhitten We downloaded the latest 'Master build' from https://github.com/microsoft/BotFramework-Composer/tree/master

@beyackle
Copy link
Contributor

beyackle commented Apr 8, 2020

@ashish315 Could you confirm that PR#2551 (this one: https://github.com/microsoft/BotFramework-Composer/pull/2551/files) is on there? That's the one that fixes this issue, and those "?" things in the field labels are no longer buttons in that one.

@ashish315
Copy link
Author

As checked i guess this is not present their. Have a look below image

image

@beyackle
Copy link
Contributor

beyackle commented Apr 8, 2020

I've rebuilt from master to double-check this myself and the screen reader reads the contents of the tooltip (followed by "group", because that's the kind of thing it's focused on). Here's a link to the video. https://microsoft-my.sharepoint.com/:v:/p/beyackle/EQg6A97M3xhKnlMH38nCmzMBV-lQoPhUVlycqfA9ockYQg?e=9pYPIh

@corinagum
Copy link
Contributor

+1 to beyackle. I have cloned the latest master and can see his fix. Please re-verify.

@ashish315
Copy link
Author

@corinagum As checked, the issue is fixed on the latest Master Bot. So, HCL is closing the issue.
Now, narrator narrates them as group with proper information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants