Skip to content

[ML] Anomaly Detection result views: ensures chart tooltips are always shown correctly#246077

Merged
alvarezmelissa87 merged 1 commit intoelastic:mainfrom
alvarezmelissa87:ml-annotation-tooltip-fix
Dec 12, 2025
Merged

[ML] Anomaly Detection result views: ensures chart tooltips are always shown correctly#246077
alvarezmelissa87 merged 1 commit intoelastic:mainfrom
alvarezmelissa87:ml-annotation-tooltip-fix

Conversation

@alvarezmelissa87
Copy link
Contributor

@alvarezmelissa87 alvarezmelissa87 commented Dec 11, 2025

Summary

This PR updates the z index for chart tooltips to ensure they are always visible.

Fixes #245852

Before:

image

After:

image

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

  • Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
  • Documentation was added for features that require explanation or tutorials
  • Unit or functional tests were updated or added to match the most common scenarios
  • If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
  • This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
  • Flaky Test Runner was used on any tests changed
  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.
@alvarezmelissa87 alvarezmelissa87 self-assigned this Dec 11, 2025
@alvarezmelissa87 alvarezmelissa87 requested a review from a team as a code owner December 11, 2025 21:03
@alvarezmelissa87 alvarezmelissa87 added :ml release_note:skip Skip the PR/issue when compiling release notes v9.3.0 labels Dec 11, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@alvarezmelissa87 alvarezmelissa87 added backport:skip This PR does not require backporting Feature:Anomaly Detection ML anomaly detection labels Dec 11, 2025
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #40 / serverless security UI Security ML Anomaly detection jobs list job creation does not show exclude frozen data tier control in wizard
  • [job] [logs] FTR Configs #61 / Visualizations - Group 3 lens app - TSVB Open in Lens Gauge "before each" hook for "should show the "Convert to Lens" menu item"

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 5.6MB 5.6MB +19.0B

cc @alvarezmelissa87

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and LGTM

@peteharverson peteharverson added release_note:fix and removed release_note:skip Skip the PR/issue when compiling release notes labels Dec 12, 2025
Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alvarezmelissa87 alvarezmelissa87 merged commit bb39d4d into elastic:main Dec 12, 2025
30 checks passed
@alvarezmelissa87 alvarezmelissa87 deleted the ml-annotation-tooltip-fix branch December 12, 2025 18:14
seanrathier pushed a commit to seanrathier/kibana that referenced this pull request Dec 15, 2025
… shown correctly (elastic#246077)

## Summary

This PR updates the z index for chart tooltips to ensure they are always
visible.

Fixes elastic#245852

Before:

<img width="636" height="148" alt="image"
src="https://github.com/user-attachments/assets/b014562b-a032-42cb-8f12-5ece47b83b8a"
/>


After:

<img width="1137" height="510" alt="image"
src="https://github.com/user-attachments/assets/ca1cabec-ed32-481c-afd6-56fe1a5640e4"
/>



### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [ ] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [ ] If a plugin configuration key changed, check if it needs to be
allowlisted in the cloud and added to the [docker
list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker)
- [ ] This was checked for breaking HTTP API changes, and any breaking
changes have been approved by the breaking-change committee. The
`release_note:breaking` label should be applied in these situations.
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [ ] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
@peteharverson peteharverson changed the title [ML] Anomaly Detection result views: ensure chart tooltips are always shown correctly Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:Anomaly Detection ML anomaly detection :ml release_note:fix v9.3.0

4 participants