Multi-link nodes, canvas UX improvements, and spritesheet node#11
Open
jackbach wants to merge 9 commits into
Open
Multi-link nodes, canvas UX improvements, and spritesheet node#11jackbach wants to merge 9 commits into
jackbach wants to merge 9 commits into
Conversation
Refactor pipeline data model to support named handles per node, object-based NodeOutput, multi-input spritesheet node, and connection validation. Unfinished — edge connection bug under investigation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Unconnected input nodes no longer block pipeline execution — they are simply skipped. Also removes auto-spawn behavior that created a new input node whenever all existing inputs had images assigned. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Node deletion is now handled exclusively via right-click context menu and keyboard shortcuts. Floating handle labels (Sheet, Data) removed for a cleaner visual style. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Previously only Delete removed nodes while both Delete and Backspace removed edges. Now both keys work consistently for both. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Right-click canvas background shows add-node menu with icons matching the top bar, separated into I/O and processing groups - Right-click node shows delete option with trash icon - Right-click edge shows delete option with trash icon - All context menus match DropdownMenu component styling - Menus flip upward/leftward when near viewport edges - Fix edge persistence: use store.edges.push() instead of addEdges() and move isValidConnection to onConnect handler only Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Migrate store from single selectedNodeId to selectedNodeIds array, sync selection via Vue Flow's getSelectedNodes, and support bulk delete from keyboard and context menu. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Dropping multiple images onto the canvas assigns them to existing input nodes (sorted top-to-bottom), then creates new input nodes for overflow. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
1 participant
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
Todo
Test plan
🤖 Generated with Claude Code