-
-
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
DataGrid: Fix empty footer row when using SelectColumn
with ShowInFooter=false
#7747
DataGrid: Fix empty footer row when using SelectColumn
with ShowInFooter=false
#7747
Conversation
Fixes an issue with the `SelectColumn` when setting `ShowInFooter` to `false`. Previously this would render a footer row (more specifically some `td`'s with the `footer-cell` CSS class) that would take up some height.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #7747 +/- ##
==========================================
- Coverage 89.78% 87.44% -2.34%
==========================================
Files 411 393 -18
Lines 11840 11641 -199
Branches 2363 2330 -33
==========================================
- Hits 10630 10179 -451
- Misses 682 950 +268
+ Partials 528 512 -16 ☔ View full report in Codecov by Sentry. |
SelectColumn
with ShowInFooter=false
SelectColumn
with ShowInFooter=false
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.
This looks good, @igotinfected. Thanks!
Ugh, I somehow missed this. I added tj to review, but didn't subscribe on updates... |
@ScarletKuro Been busy with work on my end too so thanks for the reminder! Fixing it up right now. |
@ScarletKuro Done, good to go! |
Description
Fixes an issue with the
SelectColumn
when settingShowInFooter
tofalse
.Previously this would render a footer row (more specifically some
td
's with thefooter-cell
CSS class) that would take up some height.Open to suggestions for a better solution if there is one for this case, I admit I didn't go deeper into the
FooterTemplate
rendering on theMudDataGrid
side of things.Reproduction (docs example with
ShowInFooter = false
): https://try.mudblazor.com/snippet/QOwHlvajhmmyCBhMHow Has This Been Tested?
bUnit tested
Types of changes
Previous behaviour:
firefox_FUNYj65Xl4.mp4
New behaviour:
firefox_DU3DTeB3yy.mp4
Checklist:
dev
).