Skip to content
This repository was archived by the owner on Feb 27, 2026. It is now read-only.

Fix file manager navigation, sorting, modal layout, and shared link navigation#389

Draft
Copilot wants to merge 6 commits intonextfrom
copilot/fix-file-manager-ux-issues
Draft

Fix file manager navigation, sorting, modal layout, and shared link navigation#389
Copilot wants to merge 6 commits intonextfrom
copilot/fix-file-manager-ux-issues

Conversation

Copy link
Contributor

Copilot AI commented Dec 15, 2025

📝 Description

Addresses 6 file manager UX issues: folder navigation requiring refresh, missing sort functionality, broken move modal on deep hierarchies, and shared link navigation resets.

Changes

Navigation Fix

  • Removed allFiles.length > 0 guard preventing navigation in empty folders
  • Fixed shared link circular dependency in useEffect causing unexpected navigation resets

Sorting Implementation

  • Added sort by name/date/size with ascending/descending order
  • Default sort: name ascending
  • Persists across navigation and search

Move Modal Layout

  • Converted to flex column with scrollable content area (min 200px, max 350px)
  • Fixed footer positioning independent of content height

Verification

  • Drag & drop already handles multi-select correctly
  • Bulk move operations already process all selected items

🔗 Related Issue(s)

File manager UX and functionality issues (drag & drop, multi-select, sorting, navigation)

💡 Motivation and Context

Users reported folder navigation requiring manual refresh, inability to sort files/folders, move modal breaking on deep hierarchies, and shared link navigation jumping unexpectedly to root.

🤖 Use of Artificial Intelligence (AI)

  • Yes, AI tools assisted in the development of this PR (please specify which ones and how they were used):
    • Tool(s) used: GitHub Copilot
    • Brief description of how AI contributed: Code analysis, implementation assistance, and testing guidance
  • No

🧪 How Has This Been Tested?

  • TypeScript type checking passes
  • ESLint validation passes
  • Manual verification of navigation, sorting, and modal behavior
  • Code review completed

🔄 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 to change)
  • 📚 Documentation update

✅ Checklist

  • My code follows the code style of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have rebased and/or merged on top of the latest next branch

🙏 Thank you for your contribution!

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • checkpoint.prisma.io
    • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/Palmr/Palmr/apps/server/node_modules/.pnpm/prisma@6.11.1_typescript@5.8.3/node_modules/prisma/build/child {"product":"prisma","version":"6.11.1","cli_install_type":"local","information":"","local_timestamp":"2025-12-15T18:01:12Z","project_hash":"91d7cb56","cli_path":"/home/REDACTED/work/Palmr/Palmr/apps/server/node_modules/prisma/build/index.js","cli_path_hash" sh ules�� npx tsc --noEmit --skipLibCheck (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>File manager UX and functionality issues (drag & drop, multi-select, sorting, navigation)</issue_title>
<issue_description>Hi! I jsut installed Palmr on my server. It looks great already and seems very intuitive.

After playing around with it for a while, I came across some things that could be improved. See below.


1. Drag & drop (single + multi-select)

  • Add support for dragging and dropping files and folders (e.g., moving items into other folders).
  • When multiple items are selected, allow dragging and dropping the entire selection as a batch.

2. Multi-select move action moves only one item

  • When selecting multiple files via checkmarks and choosing Move (or equivalent), only one file is moved.
  • Expected behavior: all selected items should be moved to the chosen destination folder.

3. Sorting: default + by name

  • Add the ability to sort files/folders by name.
  • Current behavior appears unsorted (or inconsistent).
  • Expected default: items are sorted by name (with optional sort controls for ascending/descending).

4. Folder navigation requires manual refresh

  • Clicking a folder updates the URL, but the UI does not actually navigate into that folder until the page is manually refreshed.
  • Expected behavior: the folder view updates immediately upon click (and stays in sync with the URL).

5. Move modal layout breaks on deep folder hierarchies

  • In the “Move item to another location” modal, when the folder hierarchy is long/deep, the buttons become misaligned/dislocated.
  • Expected behavior: the modal remains responsive and buttons stay correctly positioned (e.g., fixed footer, scrollable content area).

6. Folder color customization

  • Add the ability to assign colors to folders (for quick visual categorization).

7. Shared link navigation conflicts with root

After sharing a folder via a generated link, navigation behaves inconsistently:

  • The shared folder context appears to conflict with “root”.
  • When entering subfolders, clicking sometimes resets navigation back to the shared folder’s root instead of staying within the selected subfolder path.
  • Expected behavior: shared links should keep a consistent base path and allow stable navigation through subfolders without unexpected jumps.
    </issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 5 commits December 15, 2025 17:54
…dal layout

Co-authored-by: danielalves96 <62755605+danielalves96@users.noreply.github.com>
Co-authored-by: danielalves96 <62755605+danielalves96@users.noreply.github.com>
Co-authored-by: danielalves96 <62755605+danielalves96@users.noreply.github.com>
Co-authored-by: danielalves96 <62755605+danielalves96@users.noreply.github.com>
Co-authored-by: danielalves96 <62755605+danielalves96@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix file manager UX and functionality issues Fix file manager navigation, sorting, modal layout, and shared link navigation Dec 15, 2025
Copilot AI requested a review from danielalves96 December 15, 2025 18:11
@jr-k
Copy link

jr-k commented Dec 18, 2025

Navigation fix 💪

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.

3 participants