forked from Cloud-Pipelines/pipeline-editor
-
Notifications
You must be signed in to change notification settings - Fork 5
Remove @radix-ui/react-hover-card and refactor execution status handling #1579
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Mbeaulne
wants to merge
1
commit into
master
Choose a base branch
from
12-18-unify_status_better
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+840
−721
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
a02a1f1 to
3029633
Compare
a2650c8 to
1e3965d
Compare
7c954c4 to
949efd4
Compare
b8a7076 to
227876d
Compare
maxy-shpfy
reviewed
Dec 19, 2025
| "@radix-ui/react-checkbox": "^1.3.3", | ||
| "@radix-ui/react-collapsible": "^1.1.3", | ||
| "@radix-ui/react-dialog": "^1.1.14", | ||
| "@radix-ui/react-hover-card": "^1.1.15", |
Collaborator
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wondering if we can split PRs?
maxy-shpfy
reviewed
Dec 19, 2025
maxy-shpfy
reviewed
Dec 19, 2025
589df57 to
e1a9094
Compare
e1a9094 to
65cdf3c
Compare
65cdf3c to
adf56fd
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

Description
Enhanced status visualization with detailed tooltips for pipeline runs. The status bar now shows more granular execution states and provides tooltips with detailed breakdowns of task statuses. This improves visibility into pipeline execution states by displaying the actual execution statuses rather than simplified aggregates.
We’ll display the raw status coming from the server, but map it to a “pretty” label in the UI so we can localize later, if we need to:
Previously we grouped several “pending-like” states together (e.g., PENDING, QUEUED, WAITING, WAITING_FOR_UPSTREAM) and displayed them all as Pending.
Now, each execution state displays its own label and its own status color. Some statuses still share the same color to indicate they’re part of the same general category/type (e.g., multiple “in progress / not started yet” states use yellow).
Type of Change
Checklist
Screenshots (if applicable)
Here is a real example of my local env
Here is some test data (ignore the state on the left, I was just feeding fake data into the bar for the screenshots below, so it would contain all states)
states.mov (uploaded via Graphite)
Test Instructions
Additional Comments
This change introduces a new utility module for execution status handling and standardizes how we display and aggregate execution statuses throughout the UI.