Skip to content

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

Merged
arminmeh merged 2 commits intomui:masterfrom
arminmeh:cell-selection-event-fix
Aug 21, 2024
Merged

[DataGridPremium] Fix onCellSelectionModelChange not triggered when additional cell range is selected#14199
arminmeh merged 2 commits intomui:masterfrom
arminmeh:cell-selection-event-fix

Conversation

@arminmeh
Copy link
Contributor

Fixes #14184

Another reference is needed to detect the state change and trigger the event

@arminmeh arminmeh added type: bug It doesn't behave as expected. scope: data grid Changes related to the data grid. plan: Premium Impact at least one Premium user. feature: Cell Selection Related to the cell selection feature labels Aug 14, 2024
@mui-bot
Copy link

mui-bot commented Aug 14, 2024

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

Generated by 🚫 dangerJS against 800d54f

@cherniavskii cherniavskii changed the title [DataGridPremium] onCellSelectionModelChange not triggered when additional cell range is selected [DataGridPremium] Fix onCellSelectionModelChange not triggered when additional cell range is selected Aug 21, 2024
@arminmeh arminmeh force-pushed the cell-selection-event-fix branch from 13c0a01 to 800d54f Compare August 21, 2024 19:12
@arminmeh arminmeh enabled auto-merge (squash) August 21, 2024 19:12
@arminmeh arminmeh merged commit 7aaecf5 into mui:master Aug 21, 2024
@arminmeh arminmeh deleted the cell-selection-event-fix branch August 21, 2024 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: Cell Selection Related to the cell selection feature plan: Premium Impact at least one Premium user. scope: data grid Changes related to the data grid. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[data grid] onCellSelectionModelChange not triggered when ctrl+dragging to select additional cells

3 participants