Open
Conversation
Implements nested folder support with the following features: - Created FolderHierarchy utility to build hierarchical tree from flat folder list - Updated UI to render nested folders with proper indentation - Added "Create Subfolder" option to folder context menu - Implemented drag-and-drop support for: - Dragging conversations into nested folders - Dragging folders into other folders to create nesting - Prevention of circular references (can't drop folder into itself or descendants) - Folders automatically expand when new subfolders are created or moved into them - Visual indentation scales with folder depth for clear hierarchy display The implementation leverages the existing parentId field in the Folder model, which already supported nesting at the API level. The changes are primarily UI enhancements to display and manage the folder hierarchy. Addresses: cogwheel0#81
Previously, long-press on folders triggered the context menu, which prevented the LongPressDraggable widget from initiating drag operations. Changes: - Removed onLongPress handler from folder InkWell - Added explicit menu button (three dots icon) to folder headers - Long-press gesture now exclusively handles drag-and-drop UX flow: - Tap folder → expand/collapse - Long-press and drag folder → nest folder into another - Click menu button → show context menu (new subfolder, rename, delete)
Previously, folder counts only showed conversations, making empty folders with subfolders appear as "0", with no indication they contained nested folders. This made it unclear that clicking to expand would reveal content. Changes: - Updated folder count to include both conversations AND subfolders - Empty parent folders now show correct count (e.g., "1" if containing 1 subfolder) - Makes nested folder structure more discoverable and intuitive Example: - Before: Folder with 0 chats and 2 subfolders showed "0" - After: Same folder now shows "2"
|
Would love for this to get merged, would make Conduit feel even more powerful than most AI chat apps. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements full support for nested folders with intuitive drag-and-drop interface and visual hierarchy.
Changes
Technical Implementation
FolderHierarchyutility class to build tree structure from flat folder listDragTargetsystem for both conversation and folder dropsparentIdfield in Folder model (API already supported nesting)UI/UX Improvements
Testing
Closes #81