fix(a11y): announce when no matches are found in people-picker - #3180
Conversation
|
📖 The updated storybook is available here |
1 similar comment
|
📖 The updated storybook is available here |
|
@Raisul123 @vagpt please confirm that this fixes the narration issue |
|
📖 The updated storybook is available here |
|
📖 The updated storybook is available here |
gavinbarron
left a comment
There was a problem hiding this comment.
This is fine. In future we should look to see if we can capture the loading, no results, and results list under a single aria-live parent by refactoring the HTML that we emit.
|
This issue is still repro at oue end. URL: https://mgt.dev/next/pr/3180/?path=/docs/components-mgt-people-picker--docs MGTP-.issue.is.repro.mp4 |
|
📖 The updated storybook is available here |
|
📖 The updated storybook is available here |
|
📖 The updated storybook is available here |
|
📖 The updated storybook is available here |
|
This issue is still repro on the URL below. Please find the attachment below. URL - https://mgt.dev/next/pr/3180/?path=/docs/components-mgt-people-picker--docs Attachment: SR.is.still.not.announcing.the.message.mp4 |
|
📖 The updated storybook is available here |
|
@Raisul123 @vagpt see below, it does announce when no matches are found. 2024-05-29-13-00-01.mp4 |
|
📖 The updated storybook is available here |
|
📖 The updated storybook is available here |
|
Hi @Mnickii Attaching the video for the reference. MGTP-.issue.is.partially.fixed.mp4 |
|
📖 The updated storybook is available here |
|
Hi @Mnickii @gavinbarron, As discussed on MGTP group chat tracking "The Actual and Expected results when user types any character" related issue with Bug #3118 . Closing the issue #3117 as Currently screen reader is announcing the information “We dint fined any suggestion” shown when navigating to the 'Search for a Name' edit field. CC: @vagpt |
|
📖 The updated storybook is available here |
|
📖 The updated storybook is available here |
|
Closes #3117
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