-
Notifications
You must be signed in to change notification settings - Fork 376
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
[Dialog] Make dismiss button larger and use neutral color #2565
Conversation
- Change dismiss icon color
✅ All tests passed successfully Details on your Workflow / Core Tests page. |
Unit Tests
Details on your Workflow / Core Tests page. |
@vnbaaij Have you noticed that several unit tests are failing? |
Yes, working on that |
Summary - Unit Tests Code CoverageSummary
CoverageMicrosoft.FluentUI.AspNetCore.Components - 61.2%
|
ThreeState="false" | ||
@bind-Value="@myValue" /> | ||
); | ||
// Temporarily disable this test until we figure out how why the delay in checkbox breaks this |
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.
That's not a good idea :-)
Can you create another PR to fix that?
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 know what is causing the issue. Need to take a look at it with you when you are back. Won't be forgotten!
…2565) * - Make dialog header dismiss button a bit larger - Change dismiss icon color * Update tests * Temp disable one test
Fix #2539
In accordance with the React and v3 Web Components, the dismiss icon has been enlarged to 20px (from 16px) and the color has been changed to neutral (instead of accent)
Before
After