-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[APM] add comparison to Instances latency distribution #97710
[APM] add comparison to Instances latency distribution #97710
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
@formgeist I'm not color-blind, and I know we're going to use shapes in an upcoming revision, but I do find the grey and blue colors very hard to tell apart.
...k/plugins/apm/public/components/shared/charts/instances_latency_distribution_chart/index.tsx
Outdated
Show resolved
Hide resolved
...k/plugins/apm/public/components/shared/charts/instances_latency_distribution_chart/index.tsx
Outdated
Show resolved
Hide resolved
...k/plugins/apm/public/components/shared/charts/instances_latency_distribution_chart/index.tsx
Outdated
Show resolved
Hide resolved
Pinging @elastic/apm-ui (Team:apm) |
I agree that with these dot sizes the colors do blend together especially when they're in close vicinity. @cauemarcondes can we switch to use Another small thing; the order of the series in the chart is reversed from all the other panels. |
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
* adding comparion data to chart * fixing api test * addressing comments * refactoring
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
* adding comparion data to chart * fixing api test * addressing comments * refactoring
* adding comparion data to chart * fixing api test * addressing comments * refactoring Co-authored-by: Cauê Marcondes <[email protected]>
* adding comparion data to chart * fixing api test * addressing comments * refactoring Co-authored-by: Kibana Machine <[email protected]>
closes #97632