Skip to content

Moves node actions to menu bar Node menu#2149

Open
Mbeaulne wants to merge 1 commit into04-23-move_colour_selectorfrom
04-23-move_actions_to_menu
Open

Moves node actions to menu bar Node menu#2149
Mbeaulne wants to merge 1 commit into04-23-move_colour_selectorfrom
04-23-move_actions_to_menu

Conversation

@Mbeaulne
Copy link
Copy Markdown
Collaborator

@Mbeaulne Mbeaulne commented Apr 23, 2026

Description

Adds a Node menu to the editor menu bar that consolidates node-level actions (duplicate, delete, arrange/z-index, and unpack subgraph) into a single dropdown. This menu is context-sensitive and only appears when a node is selected.

As part of this change, the inline TaskActionsBar (with its individual DeleteTaskButton, DuplicateTaskButton, and UnpackSubgraphButton components) and the StackingControls/ZIndexEditor controls from the task and IO node detail panels have been removed. Their functionality is now accessible exclusively through the new NodeMenu in the menu bar.

The EditorMenuBar has also been moved inside the ReactFlowProvider so the NodeMenu can access the React Flow instance (e.g., getNodes) needed to compute z-index ordering.

Related Issue and Pull requests

Type of Change

  • Bug fix
  • New feature
  • Improvement
  • Cleanup/Refactor
  • Breaking change
  • Documentation update

Checklist

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

Screenshots (if applicable)

image.png

image.png

Test Instructions

  1. Open the editor and select a task node.
  2. Confirm the Node menu appears in the menu bar.
  3. Verify Duplicate Task, Delete Task, and Arrange (Move Forward, Move Backward, Bring to Front, Send to Back) all work correctly.
  4. For a subgraph task, confirm the Unpack Subgraph option appears and functions correctly.
  5. Select an input or output node and confirm the Node menu appears with only the Arrange submenu (no duplicate/delete options).
  6. Deselect all nodes and confirm the Node menu is no longer visible.

Additional Comments

The ZIndexEditor and StackingControls components may now be unused elsewhere and can be considered for removal in a follow-up cleanup.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

🎩 Preview

A preview build has been created at: 04-23-move_actions_to_menu/974a715

Copy link
Copy Markdown
Collaborator Author

Mbeaulne commented Apr 23, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@Mbeaulne Mbeaulne force-pushed the 04-23-move_actions_to_menu branch from 050e27a to cc744af Compare April 23, 2026 15:48
@Mbeaulne Mbeaulne force-pushed the 04-23-move_colour_selector branch from 333e5e5 to 45fd6a2 Compare April 23, 2026 15:48
@Mbeaulne Mbeaulne changed the title Move actions to menu Moves node actions to menu bar Node menu Apr 23, 2026
@Mbeaulne Mbeaulne marked this pull request as ready for review April 23, 2026 15:55
@Mbeaulne Mbeaulne requested a review from a team as a code owner April 23, 2026 15:55
@Mbeaulne Mbeaulne force-pushed the 04-23-move_actions_to_menu branch from cc744af to 974a715 Compare April 23, 2026 16:39
@Mbeaulne Mbeaulne force-pushed the 04-23-move_colour_selector branch from 45fd6a2 to bff8d22 Compare April 23, 2026 16:39
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