-
Notifications
You must be signed in to change notification settings - Fork 395
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
[office-ui upgrade] Move checkbox to fluentui #1465
Conversation
Signed-off-by: vinutha karanth <[email protected]>
Signed-off-by: vinutha karanth <[email protected]>
Signed-off-by: Ke Xu <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #1465 +/- ##
==========================================
+ Coverage 85.39% 89.48% +4.09%
==========================================
Files 58 84 +26
Lines 2862 3652 +790
==========================================
+ Hits 2444 3268 +824
+ Misses 418 384 -34
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
Signed-off-by: vinutha karanth <[email protected]>
…olbox into vikarant/officeUIUpgrade1
Signed-off-by: vinutha karanth <[email protected]>
I am very excited about this! Are we planning to replace other elements with fluentui elements soon, too? |
Yes, all elements will be updated in coming PRs one by one. |
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.
OK.... not that I have a lot to comment on....
* update Signed-off-by: vinutha karanth <[email protected]> * lintfix Signed-off-by: vinutha karanth <[email protected]> * update tslint Signed-off-by: Ke Xu <[email protected]> * checkbox update Signed-off-by: vinutha karanth <[email protected]> * lintfix Signed-off-by: vinutha karanth <[email protected]> Co-authored-by: Ke Xu <[email protected]>
Description
Moving checkbox to fluent UI. No UI changes, Just migration.
Checklist