Skip to content

ui: Handled the utilization metric selection state properly#5701

Merged
manojVivek merged 4 commits into
mainfrom
utilization-metric-selection-state
Jun 17, 2025
Merged

ui: Handled the utilization metric selection state properly#5701
manojVivek merged 4 commits into
mainfrom
utilization-metric-selection-state

Conversation

@manojVivek
Copy link
Copy Markdown
Contributor

@manojVivek manojVivek commented Jun 17, 2025

The utilization metrics graph selection state was not properly handled in #5696. This PR fixes that.

@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous Bot commented Jun 17, 2025

✅ Meticulous spotted 0 visual differences across 622 screens tested: view results.

Meticulous evaluated ~4 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit 5a4be6c. This comment will update as new commits are pushed.

Copy link
Copy Markdown
Contributor

@yomete yomete left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@manojVivek manojVivek merged commit 232687c into main Jun 17, 2025
39 of 40 checks passed
@manojVivek manojVivek deleted the utilization-metric-selection-state branch June 17, 2025 12:36
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.

2 participants