Skip to content

ui: Fixed metrics graph alignment to the grid#5703

Merged
manojVivek merged 3 commits into
mainfrom
metrics-graph-alignment
Jun 18, 2025
Merged

ui: Fixed metrics graph alignment to the grid#5703
manojVivek merged 3 commits into
mainfrom
metrics-graph-alignment

Conversation

@manojVivek

Copy link
Copy Markdown
Contributor

The metrics graph needed some padding to align it to the grid scale which in turn fixed the overflows on either side.

@alwaysmeticulous

alwaysmeticulous Bot commented Jun 18, 2025

Copy link
Copy Markdown

✅ Meticulous spotted visual differences in 315 of 526 screens tested, but all differences have already been approved: view differences detected.

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

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

@metalmatze metalmatze left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great. Even zooming out on the page everything sits nice and flush. 👍

@manojVivek
manojVivek merged commit 7aa32dd into main Jun 18, 2025
37 of 38 checks passed
@manojVivek
manojVivek deleted the metrics-graph-alignment branch June 18, 2025 08:45
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