-
Notifications
You must be signed in to change notification settings - Fork 310
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
fix(a11y): narrate person list position in mgt-agenda #3221
Conversation
@Raisul123 @vagpt this is ready for testing |
@Mnickii I've tested this, and it doesn't announce the position. It still says "group." Is it supposed to say, "Photo for Adele Vance, position one"? |
@musale are you testing the people list? It doesn't say group. Yes, it should announce the position of the person in the list. Try refresh and test it again. |
This issue is still repro'ing on the URL below as SR is still announcing as Group. URL - https://mgt.dev/next/pr/3221/?path=/docs/components-mgt-agenda--docs Attachments: SR.is.still.announcing.as.Group.mp4 |
📖 The updated storybook is available here |
Quality Gate failedFailed conditions |
@Mnickii |
Closes #3080
PR Type
Bugfix
Description of the changes
PR checklist
yarn build
) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)yarn setLicense
)Other information