-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
MudTabPanel: Allow icon color to be set from panel #9001
MudTabPanel: Allow icon color to be set from panel #9001
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #9001 +/- ##
==========================================
+ Coverage 89.82% 90.63% +0.80%
==========================================
Files 412 398 -14
Lines 11878 12382 +504
Branches 2364 2406 +42
==========================================
+ Hits 10670 11222 +552
+ Misses 681 621 -60
- Partials 527 539 +12 ☔ View full report in Codecov by Sentry. |
Should it be MudBlazor/src/MudBlazor/Components/Tabs/MudTabs.razor.cs Lines 137 to 142 in 1a68c71
Also needs unit test |
@djflan Thanks for the PR, it is good to merge except there is no unit test which is our policy for PRs. The reason is that any feature that has a test can not be accidentally be undone or broken. It is the reason why the library is so stable despite hundreds of collaborators. In your case, check that the color of the panel is used if unset and when set it overrides the color. |
…hub.com/djflan/MudBlazor into feat/mud-tab-panel-allow-set-icon-color
Added a unit test. Seems tricky to get the color of the icon. I had to assign 'Color.Success' and check the SVG class list for 'mud-success-text'. currently unsure if there is another way to do this. |
Hey not at my pc currently so could not test myself. Could you check if the icon is grayed out when the tab is disabled. This should be the case if no specific color is set as well as when there is a color set for the tab or inherited as you added here. If not you would need to pass down the "Disabled" property to the icon. |
…hub.com/djflan/MudBlazor into feat/mud-tab-panel-allow-set-icon-color
Added a check for disabled panel state and inherited color when disabled. Also added a unit test for this since I'm sure it will be requested. |
Thanks! |
Description
Allows MudTab Icon Color to be set from MudTabPanel (override)
How Has This Been Tested?
Visually tested using the existing 'RippleTest'
Type of Changes
Checklist
dev
).