Skip to content

Fix: allow a drop between two folders in the admin list - #133

Merged
SandObserver merged 1 commit into
mainfrom
fix/drop-between-folders
Aug 20, 2026
Merged

SandObserver merged 1 commit into
mainfrom
fix/drop-between-folders

Conversation

@SandObserver

Copy link
Copy Markdown
Owner

Summary

  • Add folderRowZone to admin-drag-logic.js, splitting a folder row into an above/into/below zone with a quarter-height edge at each end.
  • Use it in the dragover handler and in the touch place handler, so an edge zone shows drag-above/drag-below instead of drag-into.
  • Track the hovered zone in both drop paths and pass targetIsFolder only when the pointer is in the middle zone.

Testing
All green. Added 3 frontend tests for the zone split and for a reordering drop on a folder row.

Checklist

  • Stays within the project constraints: one container, no runtime dependencies, no build step (see CONTRIBUTING.md).
  • Tests added or updated for new logic, and the full suite, lint, and typechecks pass.
  • Docs updated if behavior or configuration changed.
  • CHANGELOG.md updated under Unreleased if the change is user-visible.

Split a folder row into three drop zones. The top and bottom quarter
reorder and show the insert line; the middle half still drops into the
folder. Both the pointer and the touch drag paths pass targetIsFolder
only for the middle zone.
@SandObserver SandObserver added the bug Something isn't working label Aug 20, 2026
@SandObserver
SandObserver merged commit d32cc0f into main Aug 20, 2026
4 checks passed
@SandObserver
SandObserver deleted the fix/drop-between-folders branch August 20, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant