Skip to content

ui: Small improvements#5421

Merged
metalmatze merged 3 commits into
mainfrom
ui-small-improvements
Jan 7, 2025
Merged

ui: Small improvements#5421
metalmatze merged 3 commits into
mainfrom
ui-small-improvements

Conversation

@metalmatze

Copy link
Copy Markdown
Member

Metrics Graph

The MetricsGraph Tooltip now shows the "Per Second" in one line rather than having it split in two. All that was necessary was using a  .

Old (two lines)
old-tooltip

New (one line)
new-tooltip

Query Selectors

When multiple selectors are used in the query UI, the buttons and dropdowns used to move down. This look odd and somewhat broken.

old-selectors

This is now fixed.

new-selectors

@metalmatze
metalmatze requested a review from a team as a code owner January 7, 2025 10:57
@alwaysmeticulous

alwaysmeticulous Bot commented Jan 7, 2025

Copy link
Copy Markdown

🤖 Meticulous spotted visual differences in 27 of 166 screens tested: view and approve differences detected.

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

Last updated for commit f830de3. This comment will update as new commits are pushed.

@yomete yomete left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm! Linter is complaining about an unused component only.

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