Skip to content

fix: v2: subgraph and multiselection fixes#2153

Open
camielvs wants to merge 1 commit intomasterfrom
04-23-fix_v2_subgraph_and_multiselection_fixes
Open

fix: v2: subgraph and multiselection fixes#2153
camielvs wants to merge 1 commit intomasterfrom
04-23-fix_v2_subgraph_and_multiselection_fixes

Conversation

@camielvs
Copy link
Copy Markdown
Collaborator

@camielvs camielvs commented Apr 23, 2026

Description

v2 editor fixes related to multiselect & subgraphs.

Fixes inconsistent messaging and iconography for subgraph creation: "Create Subgraph" now always uses that text + Layers icon. Both subgraph creation flows have been consolidated into one CreateSubgraphForm component.

Adds v1 editor multiselection modes to v2 editor (SHIFT + DRAG to full select & SHIFT + CMD + DRAG to partial select). We can reverse the modes->key shortcuts if we prefer.

Fixes an issue where created subgraphs would appear at arbitrary coordinates rather than in the same location of where the nodes were.

Related Issue and Pull requests

Type of Change

  • Bug fix
  • Improvement

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

subgraph creation input

image.png

subgraph creation position BEFORE

image.png

subgraph creation position AFTER

image.png

Test Instructions

Additional Comments

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

🎩 Preview

A preview build has been created at: 04-23-fix_v2_subgraph_and_multiselection_fixes/dffe0b0

@camielvs camielvs force-pushed the 04-23-fix_v2_subgraph_and_multiselection_fixes branch from e088557 to dffe0b0 Compare April 23, 2026 19:47
return { onSelectionChange };
return {
onSelectionChange,
selectionMode: metaKeyPressed ? SelectionMode.Partial : SelectionMode.Full,
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can inverse this if we prefer partial selection mode to be the default

@camielvs camielvs marked this pull request as ready for review April 23, 2026 19:58
@camielvs camielvs requested a review from a team as a code owner April 23, 2026 19:58
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.

1 participant