Skip to content

area/ui: add a new queryparam selectedSeries to query for cpu strips and flamechart#5544

Merged
metalmatze merged 2 commits into
mainfrom
utilization-context-menu
Feb 26, 2025
Merged

area/ui: add a new queryparam selectedSeries to query for cpu strips and flamechart#5544
metalmatze merged 2 commits into
mainfrom
utilization-context-menu

Conversation

@yomete

@yomete yomete commented Feb 26, 2025

Copy link
Copy Markdown
Contributor

This change ensures that clicking on a series in the utilization metrics graph highlights the series using a new selectedSeries queryParams. The previous method of using the query matchers filters away other series instead of simply just highlighting the clicked one.

Related PRs:
area/ui: add Modify the utilization metrics graph to show multiple series
area/ui: strip attributes from context menu

@yomete
yomete requested a review from a team as a code owner February 26, 2025 15:24
@alwaysmeticulous

alwaysmeticulous Bot commented Feb 26, 2025

Copy link
Copy Markdown

✅ Meticulous spotted zero visual differences across 193 screens tested: view results.

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

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

@metalmatze
metalmatze merged commit d65bd15 into main Feb 26, 2025
@yomete
yomete deleted the utilization-context-menu branch May 14, 2025 11:58
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