Skip to content

feat: ui: Add support for dashed lines for limit metrics in graphs#5640

Merged
metalmatze merged 2 commits into
mainfrom
ui-metrics-graph-limit
May 14, 2025
Merged

feat: ui: Add support for dashed lines for limit metrics in graphs#5640
metalmatze merged 2 commits into
mainfrom
ui-metrics-graph-limit

Conversation

@metalmatze
Copy link
Copy Markdown
Member

Filtered out virtual labels for unique key generation in colors. Introduced a strokeDasharray prop to allow dashed lines, applied to limit metrics for visual distinction.

@metalmatze
Copy link
Copy Markdown
Member Author

Will fix this bug, but the overall content of the PR is there, so please review. Thanks!

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!

Filtered out virtual labels for unique key generation in colors. Introduced a `strokeDasharray` prop to allow dashed lines, applied to limit metrics for visual distinction.
@metalmatze metalmatze force-pushed the ui-metrics-graph-limit branch from ee12e81 to ff2a43a Compare May 14, 2025 11:18
@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous Bot commented May 14, 2025

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

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

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

@metalmatze metalmatze enabled auto-merge May 14, 2025 11:38
@metalmatze metalmatze merged commit f0c168c into main May 14, 2025
37 checks passed
@metalmatze metalmatze deleted the ui-metrics-graph-limit branch May 14, 2025 12:27
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