Skip to content

Fix move to folder picker#87

Merged
jsgrrchg merged 4 commits into
mainfrom
fix/issue-42-move-to-folder-picker
May 15, 2026
Merged

Fix move to folder picker#87
jsgrrchg merged 4 commits into
mainfrom
fix/issue-42-move-to-folder-picker

Conversation

@jsgrrchg
Copy link
Copy Markdown
Owner

Summary

  • Adds a compact Move to Folder picker with folder search and / Root support.
  • Extends contextual move actions to notes, PDFs, generic files, folders, and mixed selections.
  • Shares move execution between context-menu actions and drag/drop so each item type keeps its correct backend path.

Root Cause

The context menu only modeled the move flow for Markdown notes, even though drag/drop and the backend already supported moving PDFs and generic vault files.

Validation

  • npm test -- src/features/vault/FileTree.test.tsx src/features/vault/fileTreeMoves.test.ts
  • npm run lint -- src/features/vault/FileTree.tsx src/features/vault/FileTree.test.tsx src/features/vault/fileTreeMoves.ts src/features/vault/fileTreeMoves.test.ts

Closes #42

@jsgrrchg
Copy link
Copy Markdown
Owner Author

Speed is an issue, takes half a second for the menu to appear.

@jsgrrchg jsgrrchg marked this pull request as ready for review May 15, 2026 05:59
@jsgrrchg
Copy link
Copy Markdown
Owner Author

Looks ready.

@jsgrrchg jsgrrchg merged commit 3ac9599 into main May 15, 2026
3 checks passed
@jsgrrchg jsgrrchg deleted the fix/issue-42-move-to-folder-picker branch May 15, 2026 06:04
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.

Move notes to context menu action needs to be expanded to all file types and refactored for better accesibility

1 participant