Skip to content

[6.x] Fix dragging assets onto folders in list view - #15676

Open
jaygeorge wants to merge 1 commit into
6.xfrom
15675-list-view-folder-drop
Open

jaygeorge wants to merge 1 commit into
6.xfrom
15675-list-view-folder-drop

Conversation

@jaygeorge

@jaygeorge jaygeorge commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Description of the Problem

In the asset browser's list view, dropping an asset onto a folder row throws Cannot read properties of undefined (reading 'find') and nothing moves. Grid view works.

What this PR Does

  • Table uses this.assets to find the dragged asset but never declared the assets prop, so it was undefined. It's declared now, the same way Grid does it.
  • Folder rows were also writing the hovered folder into draggingFolder, the same value that tracks a folder being dragged. Hovering now has its own dragOverFolder, as in Grid.
list-view-folder-drop.mp4

How to Reproduce

  1. Go to Assets, switch to list view and open a folder that has a subfolder and some assets.
  2. Drag an asset onto the subfolder's row.
  3. Nothing moves, and the error shows in the console.

- Table never declared the assets prop, so dropping an asset on a folder row threw and nothing moved
- Track the hovered row separately, rather than overwriting draggingFolder on dragover
@jaygeorge
jaygeorge added this pull request to stack #15680 October 19, 2026 14:24

This branch has not been deployed

No deployments
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.

1 participant