Skip to content

MudDataGrid: Allow disabling selection per row via criteria#11554

Merged
ScarletKuro merged 23 commits intoMudBlazor:devfrom
Anu6is:11546-MudDataGrid-DisableSelection-PerRow
Jan 9, 2026
Merged

MudDataGrid: Allow disabling selection per row via criteria#11554
ScarletKuro merged 23 commits intoMudBlazor:devfrom
Anu6is:11546-MudDataGrid-DisableSelection-PerRow

Conversation

@Anu6is
Copy link
Contributor

@Anu6is Anu6is commented Jun 25, 2025 •
edited
Loading

Description

Created a new DisabledFunc parameter in the SelectColumn to allow for conditionally disabling the select checkbox.
The following applies once disabled

  • The checkbox is disabled
  • Select on row click is ignored
  • Select all does not select the row

Additionally, the select all checkbox has been hidden from the header and footer when MultiSelection is false .

Resolves #11546
Resolves #11500

The documentation page includes an example of using this function.

How Has This Been Tested?

Visually and unit tests

Type of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation (fix or improvement to the website or code docs)

Docs:
image

Demo:

disable.mod.5.mp4

Checklist

  • The PR is submitted to the correct branch (dev).
  • My code follows the code style of this project.
  • I've added relevant tests.

@github-actions github-actions bot added the enhancement Adds a new feature or enhances existing functionality (not fixing a defect) in the main library label Jun 25, 2025
@codecov
Copy link

codecov bot commented Jun 25, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 97.61905% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 91.33%. Comparing base (1b5ac83) to head (fb99c5b).

Files with missing lines Patch % Lines
...udBlazor/Components/DataGrid/SelectColumn.razor.cs 91.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##              dev   #11554      +/-   ##
==========================================
+ Coverage   91.31%   91.33%   +0.02%     
==========================================
  Files         466      466              
  Lines       14725    14768      +43     
  Branches     2870     2871       +1     
==========================================
+ Hits        13446    13489      +43     
  Misses        636      636              
  Partials      643      643

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Anu6is Anu6is requested review from ScarletKuro and tjscience June 25, 2025 18:41
versile2
versile2 reviewed Jun 29, 2025
View reviewed changes
Copy link
Contributor

@versile2 versile2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very clean updates. While the proper null checks are in place I think those two should still be "nullable" for form and a better doc would always help others.

@versile2
Copy link
Contributor

versile2 commented Jul 1, 2025

real cool docs update. I like it.

@github-actions github-actions bot added the awaiting triage Needs maintainer review or assistance label Aug 3, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 7, 2025

Quality Gate Passed Quality Gate passed

Issues
9 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@danielchalmers danielchalmers removed the awaiting triage Needs maintainer review or assistance label Aug 17, 2025
@ProgramGeek02
Copy link

ProgramGeek02 commented Sep 12, 2025

Does this pull request has been merged?

@BigBasti123
Copy link

BigBasti123 commented Oct 21, 2025 •
edited
Loading

I have another question according to this. Maybe a bit late. But is it possible to automatically unselect a selected row, when the row is then filtered and hence hidden?

@mudbot
Copy link

mudbot bot commented Oct 21, 2025

That's an interesting question about how filtering should interact with selections. For usage questions and discussions about potential new behavior, the best places to get input from the community and maintainers are our support channels.

For similar topics or broader help, you're welcome to use GitHub Discussions or our Discord community.

I'm an AI assistant — If I missed something or made a mistake, please let me know in a reply!

This was referenced Feb 23, 2026
Closed
Closed
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Closed
Merged
Closed
Open
Closed
Closed
Closed
Merged
Closed
Closed
Merged
Closed
Open
Open
Closed
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@ScarletKuro ScarletKuro ScarletKuro approved these changes

@versile2 versile2 versile2 approved these changes

@tjscience tjscience Awaiting requested review from tjscience

Assignees

@ScarletKuro ScarletKuro

Labels

enhancement Adds a new feature or enhances existing functionality (not fixing a defect) in the main library

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support disabling row checkbox per item in MudDataGrid SelectColumn Selection Checkbox in Header Should Be Hidden when MultiSelection is False

7 participants

@Anu6is @versile2 @ProgramGeek02 @BigBasti123 @ScarletKuro @danielchalmers