-
-
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
New Component: Add MudFlexBreak component for forcing flex items onto a new line #8748
Conversation
Implements MudBlazor#6712 by adding a `MudGridBreak` component that can be used in the MudGrid in order to force items onto the next line. It's very simple to add because we already use flexbox in the grid.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #8748 +/- ##
==========================================
+ Coverage 89.82% 90.08% +0.25%
==========================================
Files 412 420 +8
Lines 11878 12178 +300
Branches 2364 2396 +32
==========================================
+ Hits 10670 10971 +301
+ Misses 681 667 -14
- Partials 527 540 +13 ☔ View full report in Codecov by Sentry. |
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.
It looks good to me! Nice work with nameof()
@danielchalmers This will even work with other flex-based components such as |
I also forgot to mention that we probably should add xml comments to |
@ScarletKuro @henon Take a look now |
Thanks! |
Description
Resolves #6712 by adding a
MudFlexBreak
component that can be used in any flexbox layout, such as MudGrid or MudStack, in order to force items onto the next line, similar to<br>
.It's very simple to add because we already use flexbox in the grid.
How Has This Been Tested?
visually
Type of Changes
Grid
Stack
Checklist
dev
).