FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ui: Make metrics graph height configurable by using a height value by yomete · Pull Request #6208 · parca-dev/parca · GitHub

ui: Make metrics graph height configurable by using a height value - #6208

Merged
yomete merged 7 commits into
mainfrom
metricss-height-props
Feb 23, 2026
Merged

ui: Make metrics graph height configurable by using a height value#6208
yomete merged 7 commits into
mainfrom
metricss-height-props

Conversation

yomete commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

Added a new metricsGraph.height field to ParcaContext so the graph's max height is configurable from context.

Added a new `metricsGraph.height field` to ParcaContext so the graph's max height is configurable from context.
yomete requested a review from a team as a code owner February 19, 2026 12:28

alwaysmeticulous Bot commented Feb 19, 2026
edited
Loading

Copy link
Copy Markdown

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

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

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

manojVivek 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Lgtm!

yomete merged commit b3669fd into main Feb 23, 2026
38 of 40 checks passed
yomete deleted the metricss-height-props branch February 23, 2026 14:21
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
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


Back | FazBrowse Home | New Git URL