-
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
[ML] Fix Single Metric Viewer & Explorer annotation table actions overflow and annotations count not matching #104955
Conversation
Pinging @elastic/ml-ui (:ml) |
jenkins, test this (had to abort for Jenkins upgrade) |
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.
Tested and LGTM
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.
Latest edits LGTM
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @qn895 |
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.
LGTM ⚡
…rflow and annotations count not matching (elastic#104955) * Fix annotations * Fix translations * Fix onclick open * Fix label/aggregations mismatch * Fix title
…rflow and annotations count not matching (elastic#104955) * Fix annotations * Fix translations * Fix onclick open * Fix label/aggregations mismatch * Fix title
…rflow and annotations count not matching (#104955) (#105140) * Fix annotations * Fix translations * Fix onclick open * Fix label/aggregations mismatch * Fix title Co-authored-by: Quynh Nguyen <[email protected]>
…rflow and annotations count not matching (#104955) (#105141) * Fix annotations * Fix translations * Fix onclick open * Fix label/aggregations mismatch * Fix title Co-authored-by: Quynh Nguyen <[email protected]>
Summary
This PR fixes a regression introduced by #101236 which causes the labels to overlap in the Explorer/Single Metric View. The disabled option was also still clickable, which leads to an unsupported page.
Before
After
Annotations table in the Explorer view
Annotations table in the job list
When Single Metric Viewer view is disabled
Also fixes #98074 by indicating more clearly that we are only showing 500 annotations.
Checklist
Delete any items that are not applicable to this PR.