Skip to content

Refactor run tools layout and add displayLabel/showTooltip props to pipeline buttons#2515

Merged
Mbeaulne merged 1 commit into
masterfrom
run-v2-tools-panel
Jul 15, 2026
Merged

Refactor run tools layout and add displayLabel/showTooltip props to pipeline buttons#2515
Mbeaulne merged 1 commit into
masterfrom
run-v2-tools-panel

Conversation

@Mbeaulne

@Mbeaulne Mbeaulne commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Description

Refactors the run tools panel in the RunView to display action buttons as a vertical list with full-width, text-labeled menu items instead of a compact inline row of icon buttons. Tooltips are now hidden for these buttons since the labels are always visible, and a displayLabel prop allows overriding the button text independently of showLabel. A showTooltip prop has been added to all relevant pipeline action buttons (CancelPipelineRunButton, ClonePipelineButton, InspectPipelineButton, RerunPipelineButton, and ViewYamlButton) to control tooltip visibility.

TooltipButton has also been updated to skip rendering the tooltip wrapper entirely when no tooltip is provided, and now accepts a wrapperClassName prop to allow styling the inner wrapper div.

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)

Test Instructions

  1. Navigate to a run in the Run View.
  2. Open the tools panel and verify the action buttons appear as a vertical list with full-width labels (e.g., "View YAML", "Inspect pipeline", "Clone pipeline", "Cancel run", "Rerun pipeline").
  3. Confirm that no tooltips appear on hover for these buttons.
  4. Verify that buttons only appear when their conditions are met (e.g., "Cancel run" only when the run is in progress and the user is the run creator, "Rerun pipeline" only when the run is complete).
  5. Confirm that pipeline action buttons used elsewhere still show tooltips and behave as expected.

Additional Comments

@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: run-v2-tools-panel/8baee87

@Mbeaulne Mbeaulne marked this pull request as ready for review July 13, 2026 21:38
@Mbeaulne Mbeaulne requested a review from a team as a code owner July 13, 2026 21:38
@Mbeaulne Mbeaulne changed the title Polish Run V2 tools panel Refactor run tools layout and add displayLabel/showTooltip props to pipeline buttons Jul 13, 2026
@Mbeaulne Mbeaulne force-pushed the run-v2-default-window-layout branch from d2595bb to 4a603a5 Compare July 13, 2026 22:14
@Mbeaulne Mbeaulne force-pushed the run-v2-tools-panel branch from 6182357 to 452ea66 Compare July 13, 2026 22:14
@Mbeaulne Mbeaulne force-pushed the run-v2-default-window-layout branch from 4a603a5 to c922def Compare July 13, 2026 22:25
@Mbeaulne Mbeaulne force-pushed the run-v2-tools-panel branch from 452ea66 to 4b791c7 Compare July 13, 2026 22:25
Comment thread src/components/PipelineRun/components/CancelPipelineRunButton.tsx Outdated
Comment thread src/components/shared/Buttons/ViewYamlButton.tsx Outdated

Mbeaulne commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator Author

Merge activity

  • Jul 15, 5:17 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jul 15, 5:23 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jul 15, 5:26 PM UTC: @Mbeaulne merged this pull request with Graphite.

@Mbeaulne Mbeaulne changed the base branch from run-v2-default-window-layout to graphite-base/2515 July 15, 2026 17:17
@Mbeaulne Mbeaulne changed the base branch from graphite-base/2515 to master July 15, 2026 17:21
@Mbeaulne Mbeaulne force-pushed the run-v2-tools-panel branch from 453c3e5 to 8baee87 Compare July 15, 2026 17:22
@Mbeaulne Mbeaulne merged commit 99e71c9 into master Jul 15, 2026
17 checks passed
@Mbeaulne Mbeaulne deleted the run-v2-tools-panel branch July 15, 2026 17:26
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.

3 participants