Skip to content

Conversation

@Mbeaulne
Copy link
Collaborator

@Mbeaulne Mbeaulne commented Dec 5, 2025

Description

Added copy functionality to pipeline names in the UI. The CopyText component now has an alwaysShowButton prop that keeps the copy button visible at all times rather than only on hover. This enhancement has been implemented in both the Pipeline Details and Run Details views, allowing users to easily copy pipeline names.

Type of Change

  • Improvement
  • New feature

Checklist

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

Screenshots (if applicable)

Before:Screenshot 2025-12-05 at 1.47.50 PM.png

After

Screenshot 2025-12-05 at 1.47.25 PM.png

Bef

Test Instructions

  1. Navigate to a pipeline in the editor view
  2. Verify the copy button is always visible next to the pipeline name
  3. Click the copy button and confirm the pipeline name is copied to clipboard
  4. Navigate to a pipeline run view
  5. Verify the same copy functionality works for the pipeline name in this view

Additional Comments

This change improves user experience by making it more obvious that pipeline names can be copied, rather than requiring users to discover this functionality by hovering.

Copy link
Collaborator Author

Mbeaulne commented Dec 5, 2025

@Mbeaulne Mbeaulne marked this pull request as ready for review December 5, 2025 16:41
@Mbeaulne Mbeaulne force-pushed the 12-05-adds_copy_to_other_titles branch from 8ab89de to 8fe8f83 Compare December 5, 2025 18:45
@Mbeaulne Mbeaulne force-pushed the 12-05-adds_copy_to_other_titles branch from 8fe8f83 to b890a7c Compare December 5, 2025 19:15
@Mbeaulne Mbeaulne force-pushed the 12-05-adds_copy_to_other_titles branch from b890a7c to b9365ea Compare December 5, 2025 19:25
@Mbeaulne Mbeaulne mentioned this pull request Dec 5, 2025
8 tasks
@Mbeaulne Mbeaulne force-pushed the 12-05-adds_copy_to_other_titles branch 2 times, most recently from 5f1ed68 to da6b344 Compare December 5, 2025 20:14
@Mbeaulne Mbeaulne force-pushed the 12-05-adds_copy_to_other_titles branch 2 times, most recently from 60b3449 to fd8380e Compare December 5, 2025 21:28
@Mbeaulne Mbeaulne requested a review from camielvs December 5, 2025 21:34
@Mbeaulne Mbeaulne changed the base branch from fix_long_titles to graphite-base/1477 December 5, 2025 22:01
@Mbeaulne Mbeaulne force-pushed the 12-05-adds_copy_to_other_titles branch from fd8380e to 6520481 Compare December 5, 2025 22:02
@graphite-app graphite-app bot changed the base branch from graphite-base/1477 to master December 5, 2025 22:03
@graphite-app
Copy link

graphite-app bot commented Dec 5, 2025

Merge activity

  • Dec 5, 10:03 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.
  • Dec 6, 3:16 PM UTC: @camielvs merged this pull request with Graphite.

@Mbeaulne Mbeaulne force-pushed the 12-05-adds_copy_to_other_titles branch from 6520481 to 5697ef3 Compare December 6, 2025 15:14
@camielvs camielvs merged commit 76f0c28 into master Dec 6, 2025
5 checks passed
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