-
-
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
MudForm: Add Spacing
property
#8880
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #8880 +/- ##
==========================================
+ Coverage 89.82% 90.18% +0.35%
==========================================
Files 412 423 +11
Lines 11878 12275 +397
Branches 2364 2407 +43
==========================================
+ Hits 10670 11070 +400
+ Misses 681 670 -11
- Partials 527 535 +8 ☔ View full report in Codecov by Sentry. |
Spacing
propertySpacing
property
LGTM, just a test is missing. That check for the application of the class |
@henon Test added. Do you want me to squeeze in the spacing unification before preview2? |
No hurry. We'll do as many previews as we need. |
Description
Turns the MudForm into a flex display and uses that to add a
gap-*
class based directly on a newSpacing
property.Resolves #8859.
How Has This Been Tested?
visually
Type of Changes
video2.mp4
Checklist
dev
).