[ML] Add tooltip for interval in Anomaly Explorer#97944
[ML] Add tooltip for interval in Anomaly Explorer#97944lcawl merged 2 commits intoelastic:masterfrom
Conversation
lcawl
added
enhancement
|
Pinging @elastic/ml-ui (:ml) |
| <EuiToolTip | ||
| content={i18n.translate('xpack.ml.explorer.intervalTooltip', { | ||
| defaultMessage: | ||
| 'Show all anomalies or only the highest severity anomaly for each hour or day.', |
There was a problem hiding this comment.
Is it worth trying to explain what the auto value? (Shows by hour or day depending on the selected time span - uses hour if less than 2 days selected, otherwise uses day).
There was a problem hiding this comment.
I'm not sure that the detail of auto is needed.
One observation ... it is a better experience if you summarise your anomalies, hence the default is either hr/day. By leading with "Show all" are we encouraging "Show all"? .. because we should be encouraging to view as summarised.
There was a problem hiding this comment.
Thanks for the feedback! I've altered the text in 709de27
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
peteharverson
left a comment
peteharverson
left a comment
There was a problem hiding this comment.
Edited text LGTM.
Is it worth adding this tooltip into the Single Metric Viewer as part of this PR? If so, I think you just need to add this same code into ml/public/application/timeseriesexplorer/timeseriesexplorer.js
Good point! I've created #98174 |
5 participants
Summary
This PR adds a tooltip for the "Interval" selector in the Anomaly Explorer.