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
Remove unused Color.Surface
#8889
base: dev
Are you sure you want to change the base?
Conversation
I tried to use this color in the button expecting to see the Surface style from the color palette but instead saw the same style as the default button because the color loops don't include surface: MudBlazor/src/MudBlazor/Styles/components/_button.scss Lines 176 to 193 in d0b0545
as it is not defined in
nor does it have text, darken, hover, etc MudBlazor/src/MudBlazor/Themes/Models/Palette.cs Lines 159 to 162 in d0b0545
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #8889 +/- ##
==========================================
+ Coverage 89.82% 90.19% +0.36%
==========================================
Files 412 423 +11
Lines 11878 12274 +396
Branches 2364 2407 +43
==========================================
+ Hits 10670 11070 +400
+ Misses 681 669 -12
- Partials 527 535 +8 ☔ View full report in Codecov by Sentry. |
Color.Surface
Color.Surface
Description
I believe the Color enum member
Surface
(not the palette color) should not exist because:Color.Default
(try it on a MudButton for example).Maybe I have missed something fundamentally useful about it.
How Has This Been Tested?
visually
Type of Changes
Checklist
dev
).