Skip to content

Conversation

@SalvatorePreviti
Copy link
Contributor

@SalvatorePreviti SalvatorePreviti commented Apr 29, 2025

Enable managed drag and drop for tree data with parentId.

Changes included:

  • split the logic to set the parent in the TreeData with parentId strategy
    • We use the "treeNode" field in the RowNode as a slot to store the new tree parent to set during processing
    • in this way we don't reload the parent if there is just a drag and drop event when regrouping after a drag and drop
      • This logic will be reused after when implementing drag and drop with children, switching from the current tree data with children node manager to a common TreeStrategy
    • Fix all the code and types around it
  • New drag and drop logic in RowDragFeature that correctly handle switching tree parent and invokes grouping to finalize tree construction

@SalvatorePreviti SalvatorePreviti marked this pull request as ready for review May 13, 2025 11:48
@gportela85 gportela85 merged commit ba0ecde into latest May 13, 2025
13 checks passed
@gportela85 gportela85 deleted the AG-14392-treedata-parentid-drag-drop-logic branch May 13, 2025 20:32
@SalvatorePreviti SalvatorePreviti restored the AG-14392-treedata-parentid-drag-drop-logic branch May 15, 2025 14:14
@SalvatorePreviti SalvatorePreviti deleted the AG-14392-treedata-parentid-drag-drop-logic branch May 15, 2025 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants