-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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] Transaction duration histogram buckets without samples are clickable #98540
[APM] Transaction duration histogram buckets without samples are clickable #98540
Conversation
Pinging @elastic/apm-ui (Team:apm) |
@cauemarcondes Thanks for adding the message. I should have provided guidance earlier, but can we make it so the message is displayed below the series label and value, similar to have what we show in the Instances distribution chart; So instead of the "click to filter by instance" we'd write the "No samples available" perhaps with an |
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.
Messaging is good - but left a small note about aligning with how we display small messages in the chart tooltips, perhaps we can make it consistent here as well.
x-pack/plugins/apm/public/components/app/transaction_details/Distribution/index.tsx
Outdated
Show resolved
Hide resolved
💚 Build SucceededMetrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
…kable (elastic#98540) * adding no samples label on distribution chart * adding custom tooltip * addressing PR comments
…kable (elastic#98540) * adding no samples label on distribution chart * adding custom tooltip * addressing PR comments
…kable (elastic#98540) * adding no samples label on distribution chart * adding custom tooltip * addressing PR comments
Tests OK. Tooltips without samples are not clickable (though it would be nice if we could make it |
closes #94684