microsoft-graph-toolkit icon indicating copy to clipboard operation
microsoft-graph-toolkit copied to clipboard

[Accessibility] [Microsoft Graph Toolkit Playground - mgt-people-picker - People Picker]: Ensures every id attribute value of active elements is unique.[Sev3][WCAG4.1.1]

Open SrujanaAnaganti opened this issue 2 years ago • 1 comments

Test Environment: OS Build: Windows 11 Version: 21H2 (OS Build 22000.318) Browser: Edge Version 96.0.1054.43 (Official build) (64-bit) URL: https://mgt.dev/next Screen reader: Narrator Tool: Accessibility insights for web

Repro Steps: Open the above URL and login with valid credentials. 'Microsoft Graph Toolkit Playground' Overview page will get displayed. Navigate to 'mgt-people-picker - People Picker' and activate it. Navigate to 'Edit field' and start typing a name, Suggestion list will get displayed. Now run AI tool and verify the issue that every id attribute value of active elements is unique or not.

Actual Result: Id attribute value defined for 'Suggested list items' in 'People picker' page are not unique.

Expected Result: Ensures every id attribute value of active elements is unique for 'Suggested list items' in 'People picker' page.

User Impact: Users who depend on assistive technologies (screen readers) will face the difficulty when multiple active, focusable elements share the same id attribute, are likely to act only on the first and ignore the others. As a consequence, both functionality and accessibility can be degraded.

Note: Issue is observed for similar kind of controls through out the URL.

WCAG Reference: https://www.w3.org/TR/UNDERSTANDING-WCAG20/ensure-compat-parses.html?msclkid=29ae9875b26a11eca32e5e2893b26a12

WCAG4 1 1_Ensures every id attribute value of active elements is unique

SrujanaAnaganti avatar Apr 02 '22 09:04 SrujanaAnaganti

Hello SrujanaAnaganti, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌

ghost avatar Apr 02 '22 09:04 ghost

Hello @Laxmidurga I have a pr for this here

musale avatar Sep 07 '22 10:09 musale

Hey @musale, can i verify the issue in the below mentioned UI as am not able to find any URL to verify in attached PR. https://mgt.dev/next/?path=/story/components-mgt-people-picker--people-picker

Laxmidurga avatar Sep 07 '22 12:09 Laxmidurga

@Laxmidurga the PR is updated for review! :)

musale avatar Sep 08 '22 10:09 musale

@musale @sebastienlevert As verified, issue does not repro in the below URL. Hence, closing URL: https://mgt.dev/next/pr/1842/?path=/story/components-mgt-people-picker--people-picker

Laxmidurga avatar Sep 08 '22 10:09 Laxmidurga