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
feat: add optional person-card to people picker #2900
Conversation
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
2 similar comments
📖 The updated storybook is available here |
📖 The updated storybook is available here |
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
Something weird is happening here. The Sandbox URL is now wrong when using this PR. Did we make changes to the way we build or to the way we use this value? Also, the fallback value is not working anymore I think, it should be |
packages/mgt-components/src/components/mgt-people-picker/mgt-people-picker.scss
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-people-picker/mgt-people-picker.ts
Outdated
Show resolved
Hide resolved
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
🚀 New react-contoso sample application deployed here |
🚀 New react-contoso sample application deployed here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a missing Docs PR for this attribute that is being added.
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
🚀 New react-contoso sample application deployed here |
1 similar comment
🚀 New react-contoso sample application deployed here |
📖 The updated storybook is available here |
|
📖 The updated storybook is available here |
Closes #1940
Closes #2903
PR Type
Feature
Description of the changes
Add optional person-card to people-picker
Fixes theme colors
PR checklist
yarn build
) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)yarn setLicense
)Other information