Skip to content

[Sidenav] Keep primary item click from opening the hover popover - #290905

Open
Dosant wants to merge 1 commit into
elastic:mainfrom
Dosant:d/2026-09-14-sidenav-hover-click
Open

[Sidenav] Keep primary item click from opening the hover popover#290905
Dosant wants to merge 1 commit into
elastic:mainfrom
Dosant:d/2026-09-14-sidenav-hover-click

Conversation

@Dosant

@Dosant Dosant commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #288640.

Summary

I noticed that clicking on the primary item may open an unwanted hover popover, which gets in the way. I decided to suppress it while making sure that keyboard navigation and clicks still work.

Screen.Recording.2026-09-14.at.16.48.59.mov

A mousedown on Dashboards cancels the pending hover-open and holds the popover closed until mouseleave, so recents cannot reopen on the click's focus. Enter still opens the submenu.
@Dosant Dosant added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// labels Sep 14, 2026
@infra-vault-gh-plugin-prod

Copy link
Copy Markdown
🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!
  • Click to trigger kibana-entity-store-performance-from-pr for this PR!
  • Click to trigger kibana-storybooks-from-pr for this PR!

@Dosant Dosant changed the title Keep Dashboards click from opening the recents hover [Sidenav] Keep primary item click from opening the hover popover Sep 14, 2026
@Dosant
Dosant marked this pull request as ready for review September 14, 2026 14:59
@Dosant
Dosant requested a review from a team as a code owner September 14, 2026 14:59
@Dosant
Dosant requested a review from acstll September 14, 2026 14:59
@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

Pinging @elastic/appex-sharedux (Team:SharedUX)

@weronikaolejniczak weronikaolejniczak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 🟢

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 release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t//

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants