feat: update Todo component to new Fluent designs #1967
Conversation
|
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:
|
Mnickii
force-pushed
the
mnickii/todo-fluent-update
branch
from
767a516
to
04f050c
Compare
|
The updated storybook is available here |
1 similar comment
|
The updated storybook is available here |
|
The updated storybook is available here |
|
The updated storybook is available here |
|
@Mnickii I see there are 3 items left that are design related.
Once these are completed, I think we will be good to close this issue! 🎉 |
|
The updated storybook is available here |
|
@sebastienlevert the checkmark issues have been fixed by shifting to use fluent-checkbox rather than fluent-radio that doesn't allow selection of multiple items. |
|
The updated storybook is available here |
There was a problem hiding this comment.
@Mnickii all seems great except the hover effects on the hovered task. I don't see any changes. Besides that, we're good to go!
ghost
added
the
|
The updated storybook is available here |
|
@sebastienlevert This whole time, I missed a small typo 😭 |
|
The updated storybook is available here |
Closes #1710
Closes #2082
Closes #2084
PR Type
Feature
Description of the changes
Updates Todo component to new fluent design
Includes new mgt-picker and it's theme updates
PR checklist
yarn build) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)yarn setLicense)Other information
Requires #2048 to adequately test Task lists and Todo tasks on storybook.