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: announce teams channel results when you type #2561
Conversation
Signed-off-by: Martin Musale <martinmusale@microsoft.com>
Signed-off-by: Martin Musale <martinmusale@microsoft.com>
Signed-off-by: Martin Musale <martinmusale@microsoft.com>
…raph-toolkit into bug-2292
Thank you for creating a Pull Request @gavinbarron. This is a checklist for the PR reviewer(s) to complete before approving and merging this PR:
|
The updated storybook is available here |
Adding this here: @microsoft-graph-toolkit-write this should work BUT I have found an issue in handling the state of component. Visually, everything looks like it's working correctly. Using narrator, other bugs come up. Repro 1
This is a different experience in the people-picker. Any ideas to fix this experience are very welcome 😄 |
So what I think is happening is that as the element gets filtered elements are removed from the DOM. On the first keystroke the tree gets expanded and the child nodes rendered, which is an additive change. Let's try adding aria-relevant="all" to have deletions trigger the live behavior and aria-atomic="true" to ensure that the whole tree is announced when we add the children. |
The updated storybook is available here |
The updated storybook is available here |
…lues in the dropdown Signed-off-by: Martin Musale <martinmusale@microsoft.com>
…toolkit into bug-2292
The updated storybook is available here |
packages/mgt-components/src/components/mgt-teams-channel-picker/mgt-teams-channel-picker.ts
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-teams-channel-picker/mgt-teams-channel-picker.ts
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-teams-channel-picker/mgt-teams-channel-picker.ts
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-teams-channel-picker/mgt-teams-channel-picker.ts
Outdated
Show resolved
Hide resolved
The updated storybook is available here |
1 similar comment
The updated storybook is available here |
Hi @gavinbarron. This issue is fixed partially as on typing random text or incorrect data instead of 'We didn't find any matches' it is announcing the information about the default list item i.e. 'HR Taskforce, Business Development and X1050 Launch Team' which is irrelevant for the user. Simply screen reader should announce the error message only to the user as 'We didn't find any matches'. Test Environment: Attachment: MGTP.-.Issue.is.partially.fixed.mp4 |
The updated storybook is available here |
The updated storybook is available here |
Removes lostFocus guards Adds aria-atomic and aria-relevant attributes to cater for changing values in the dropdown Adding aria-live and other roles to no results template to ensure annoucing Shortened people picker placeholder to ensure it doesn't overflow with increased text spacing --------- Signed-off-by: Martin Musale <martinmusale@microsoft.com> Co-authored-by: Martin Musale <martinmusale@microsoft.com>
Closes #2292
PR Type
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
Replacement for #2539