microsoftgraph / microsoft-graph-toolkit

Authentication Providers and UI components for Microsoft Graph 🦒
https://docs.microsoft.com/graph/toolkit/overview
Other
939 stars 298 forks source link

fix: keyboard focus behavior on modal interactions in mgt-people-picker story #3192

Closed Mnickii closed 4 months ago

Mnickii commented 4 months ago

Closes #3123

PR Type

Bugfix

Description of the changes

PR checklist

Other information

microsoft-github-policy-service[bot] commented 4 months ago

Thank you for creating a Pull Request @Mnickii.

This is a checklist for the PR reviewer(s) to complete before approving and merging this PR:

github-actions[bot] commented 4 months ago

📖 The updated storybook is available here

github-actions[bot] commented 4 months ago

📖 The updated storybook is available here

musale commented 4 months ago

@vagpt @Raisul123 this is ready for review.

github-actions[bot] commented 4 months ago

📖 The updated storybook is available here

github-actions[bot] commented 4 months ago

📖 The updated storybook is available here

vagpt commented 4 months ago

@vagpt @Raisul123 this is ready for review.

This issue is now working as expected on the URL below hence we are good to close the issue related to this PR.

URL: https://mgt.dev/next/pr/3192/?path=/story/components-mgt-people-picker-properties--single-select-mode

Attachment:

https://github.com/microsoftgraph/microsoft-graph-toolkit/assets/95913748/35d4c168-3036-4443-9e24-b5940672e81d

github-actions[bot] commented 4 months ago

📖 The updated storybook is available here

github-actions[bot] commented 4 months ago

Code Coverage

Package Line Rate Branch Rate Complexity Health
mgt-components.src.components 86% 75% 0
mgt-components.src.components.mgt-contact 68% 83% 0
mgt-components.src.components.mgt-file 62% 100% 0
mgt-components.src.components.mgt-file-list 46% 100% 0
mgt-components.src.components.mgt-file-list.mgt-file-upload 49% 86% 0
mgt-components.src.components.mgt-get 22% 100% 0
mgt-components.src.components.mgt-messages 68% 100% 0
mgt-components.src.components.mgt-organization 47% 100% 0
mgt-components.src.components.mgt-person 82% 100% 0
mgt-components.src.components.mgt-person-card 77% 49% 0
mgt-components.src.components.mgt-picker 80% 100% 0
mgt-components.src.components.mgt-profile 40% 100% 0
mgt-components.src.components.mgt-tasks-base 87% 100% 0
mgt-components.src.components.mgt-theme-toggle 100% 100% 0
mgt-components.src.components.mgt-todo 79% 100% 0
mgt-components.src.components.sub-components.mgt-flyout 78% 53% 0
mgt-components.src.components.sub-components.mgt-spinner 100% 100% 0
mgt-components.src.graph 38% 78% 0
mgt-components.src.styles 92% 80% 0
mgt-components.src.utils 79% 27% 0
mgt-element.dist.es6.components.src.components 72% 73% 0
mgt-element.dist.es6.mock.src.mock 91% 73% 0
mgt-element.dist.es6.providers.src.providers 85% 76% 0
mgt-element.dist.es6.src 91% 80% 0
mgt-element.dist.es6.utils.src.utils 68% 71% 0
mgt-element.src 93% 40% 0
mgt-element.src.components 78% 100% 0
mgt-element.src.mock 81% 56% 0
mgt-element.src.providers 84% 100% 0
mgt-element.src.utils 72% 91% 0
Summary 67% (14551 / 21559) 74% (519 / 699) 0