-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Closed
Closed
Copy link
Labels
feature: ReorderingRelated to the data grid Reordering featureRelated to the data grid Reordering featureplan: ProImpact at least one Pro user.Impact at least one Pro user.scope: data gridChanges related to the data grid.Changes related to the data grid.type: bugIt doesn't behave as expected.It doesn't behave as expected.
Description
Steps to reproduce
Steps:
- Open this link to live example: https://stackblitz.com/edit/qsyrrzxn?file=src%2FDemo.tsx
- Try to drag a row over any cell other than row reorder cell (with reorder icon)
- Notice dropping doesn't reorder the row
Current behavior
It is not possible to perform row reorder by dropping on a cell which has cell reorder disabled.
Expected behavior
It should be possible to reorder anywhere in the Data Grid
Context
Discovered while working on #18251
Here's the culprit:
const disableDragEvents = !(canReorderColumn || (isReorderCell && canReorderRow)); |
We have to introduce some kind of state which gets set on rowDragStart
and gets cleared on rowDragEnd
.
Your environment
Not relevant
Search keywords: reorder, row reorder
cherniavskii
Metadata
Metadata
Assignees
Labels
feature: ReorderingRelated to the data grid Reordering featureRelated to the data grid Reordering featureplan: ProImpact at least one Pro user.Impact at least one Pro user.scope: data gridChanges related to the data grid.Changes related to the data grid.type: bugIt doesn't behave as expected.It doesn't behave as expected.