-
Notifications
You must be signed in to change notification settings - Fork 283
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
feature: Update agenda component to the fluent UI spec #1867
Conversation
Thank you for creating a Pull Request @musale. This is a checklist for the PR reviewer(s) to complete before approving and merging this PR:
|
The updated storybook is available here |
…lements in small viewports
The updated storybook is available here |
38ed11e
to
63bbe9f
Compare
The updated storybook is available 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.
I've yet to do a good review of the appearance, but there's a few little things that stood out.
packages/mgt-components/src/components/mgt-agenda/mgt-agenda.ts
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-agenda/mgt-agenda.ts
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-people/mgt-people.ts
Outdated
Show resolved
Hide resolved
The updated storybook is available here |
…flow at the bottom
Co-authored-by: Gavin Barron <gavinbarron@microsoft.com>
…crosoft-graph-toolkit into musale/fluentui/agenda
The updated storybook is available here |
Co-authored-by: Gavin Barron <gavinbarron@microsoft.com>
The updated storybook is available here |
The updated storybook is available here |
The updated storybook is available here |
The updated storybook is available 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.
🚀
The updated storybook is available here |
The updated storybook is available here |
The updated storybook is available here |
Closes #1624
PR Type
Description of the changes
null
people values that fail when you try to get theid
.--event-row-gap
that allows a developer to customize the size between two event elements. This property deprecates the--event-margin
property.PR checklist
yarn build
) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)yarn setLicense
)Other information