Skip to content
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

[DataGridPremium] Fix onCellSelectionModelChange not triggered when additional cell range is selected #14650

Merged
merged 4 commits into from
Sep 18, 2024

Conversation

arminmeh
Copy link
Contributor

@arminmeh arminmeh commented Sep 17, 2024

Based on #14199

Related to #14184

@arminmeh arminmeh added bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! v6.x plan: Premium Impact at least one Premium user feature: Cell Selection Related to the cell selection feature labels Sep 17, 2024
@mui-bot
Copy link

mui-bot commented Sep 17, 2024

Deploy preview: https://deploy-preview-14650--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 0412a0a

@MBilalShafi MBilalShafi added the cherry-pick The PR was cherry-picked from the newer alpha/beta/stable branch label Sep 17, 2024
@@ -31,7 +31,7 @@ firefox 78
ios_saf 12.2
kaios 2.5
op_mini all
op_mob 73
op_mob 80
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@arminmeh arminmeh merged commit e58e094 into mui:v6.x Sep 18, 2024
14 of 16 checks passed
@arminmeh arminmeh deleted the cell-selection-event-fix-v6 branch September 18, 2024 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work cherry-pick The PR was cherry-picked from the newer alpha/beta/stable branch component: data grid This is the name of the generic UI component, not the React module! feature: Cell Selection Related to the cell selection feature plan: Premium Impact at least one Premium user v6.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants