Skip to content

feat: v2 - task details context panel facelift#2186

Open
maxy-shpfy wants to merge 1 commit intomasterfrom
04-30-feat_v2_-_task_details_context_panel_facelift
Open

feat: v2 - task details context panel facelift#2186
maxy-shpfy wants to merge 1 commit intomasterfrom
04-30-feat_v2_-_task_details_context_panel_facelift

Conversation

@maxy-shpfy
Copy link
Copy Markdown
Collaborator

@maxy-shpfy maxy-shpfy commented May 1, 2026

Description

The Task Details panel has been redesigned to replace the collapsible "Arguments" and "Configuration" sections with a tabbed interface, making the panel cleaner and easier to navigate. A new "Actions" section has been added at the bottom of the panel, consolidating task-level actions (delete, duplicate, and unpack subgraph) alongside stacking controls for managing z-index.

The OutputsSection has been removed from the Arguments tab, and the arguments view now shows only the TaskArgumentsEditor directly. Three new action button components have been extracted into their own files (DeleteTaskButton, DuplicateTaskButton, UnpackSubgraphButton) and composed into a new TaskActionsBar component. When a focused argument is detected, the panel automatically switches to the "arguments" tab.

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)

Before After
image.png
image.png
image.png

Test Instructions

  1. Open the editor and select a task node to open the Task Details panel.
  2. Verify that the "Arguments" and "Configuration" tabs are present and switch correctly.
  3. Confirm that clicking an argument from the graph automatically switches to the "Arguments" tab.
  4. Scroll to the bottom of the panel and verify the "Actions" section displays the stacking controls and the appropriate action buttons (duplicate, delete, and unpack subgraph if applicable).
  5. Test each action button to confirm delete, duplicate, and unpack subgraph behave correctly with appropriate success/error notifications.

Additional Comments

The UnpackSubgraphButton only renders when the task's component implementation is a graph implementation, preserving the existing conditional display behavior.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

🎩 Preview

A preview build has been created at: 04-30-feat_v2_-_task_details_context_panel_facelift/13bbd40

@Mbeaulne Mbeaulne force-pushed the 04-23-rename_pipeline_and_task_nodes branch 2 times, most recently from a6bc1c0 to fcfc5c0 Compare May 1, 2026 20:14
@Mbeaulne Mbeaulne force-pushed the 04-30-feat_v2_-_task_details_context_panel_facelift branch from 93144e7 to 1569aa2 Compare May 1, 2026 20:14
@Mbeaulne Mbeaulne changed the base branch from 04-23-rename_pipeline_and_task_nodes to graphite-base/2186 May 1, 2026 20:18
@Mbeaulne Mbeaulne force-pushed the graphite-base/2186 branch from fcfc5c0 to 307b65e Compare May 1, 2026 20:19
@Mbeaulne Mbeaulne force-pushed the 04-30-feat_v2_-_task_details_context_panel_facelift branch from 1569aa2 to 194c9a0 Compare May 1, 2026 20:19
@graphite-app graphite-app Bot changed the base branch from graphite-base/2186 to master May 1, 2026 20:19
@Mbeaulne Mbeaulne force-pushed the 04-30-feat_v2_-_task_details_context_panel_facelift branch from 194c9a0 to 255ef17 Compare May 1, 2026 20:19
@maxy-shpfy maxy-shpfy force-pushed the 04-30-feat_v2_-_task_details_context_panel_facelift branch from 255ef17 to 4ae13a8 Compare May 5, 2026 01:52
@maxy-shpfy maxy-shpfy force-pushed the 04-30-feat_v2_-_task_details_context_panel_facelift branch from 4ae13a8 to 13bbd40 Compare May 6, 2026 00:02
@maxy-shpfy maxy-shpfy marked this pull request as ready for review May 6, 2026 00:04
@maxy-shpfy maxy-shpfy requested a review from a team as a code owner May 6, 2026 00:04
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