Skip to content

TreeView: Drag & Drop support#6537

Merged
stsrki merged 25 commits intomasterfrom
dev/treeview-dragdrop-support
Apr 29, 2026
Merged

TreeView: Drag & Drop support#6537
stsrki merged 25 commits intomasterfrom
dev/treeview-dragdrop-support

Conversation

@njannink
Copy link
Copy Markdown
Collaborator

Description

Closes #6323

How Has This Been Tested?

Types 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 not to work as expected)

Checklist:

  • The PR is submitted to the correct branch (master for dev, rel-1.x for maintenance).
  • My code follows the code style of this project.
  • I've added relevant tests.

Copy link
Copy Markdown
Collaborator

@stsrki stsrki left a comment

Choose a reason for hiding this comment

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

API-wise looks fine. Still need to run it.

Comment thread Source/Extensions/Blazorise.TreeView/Internal/_TreeViewNodeContent.razor.cs Outdated
Comment thread Source/Extensions/Blazorise.TreeView/TreeView.razor.cs Outdated
Comment thread Source/Extensions/Blazorise.TreeView/TreeView.razor.cs Outdated
Comment thread Source/Extensions/Blazorise.TreeView/Internal/_TreeViewNode.razor.cs Outdated
Comment thread Source/Extensions/Blazorise.TreeView/Styles/blazorise.treeview.scss Outdated
@stsrki
Copy link
Copy Markdown
Collaborator

stsrki commented Apr 17, 2026

I tested it. Overall, it works fine.

I noticed one issue. If you drag a node into a childless node, the target node now has child nodes. But then, if you move that same node from its parent, it still shows the expand icon even though no child nodes are available. Refreshing the tree hides the expand icon.

@njannink
Copy link
Copy Markdown
Collaborator Author

yes I saw it aswell still working on that. There are still some issues with refresh and ObservableCollections it seems.

@njannink
Copy link
Copy Markdown
Collaborator Author

Moved dragdrop logic to a seperate helper class and resolved all comments.

@njannink njannink changed the title [WIP] TreeView: Add drag+drop support TreeView: Add drag+drop support Apr 27, 2026
Comment thread Demos/Blazorise.Demo/Pages/Tests/TreeViewPage.razor.cs Outdated
@stsrki stsrki changed the title TreeView: Add drag+drop support TreeView: Drag & Drop support Apr 29, 2026
@stsrki stsrki merged commit d8067c6 into master Apr 29, 2026
2 checks passed
@stsrki stsrki deleted the dev/treeview-dragdrop-support branch April 29, 2026 09:11
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TreeView: Drag & Drop

2 participants