Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

ryankeairns
Copy link
Contributor

@ryankeairns ryankeairns commented Oct 7, 2025

Closes #8931

Summary

Primary change

Update EuiPopover defaults for modern popover appearance

  • hasArrow: false
  • offset: 4
  • anchorPosition: downLeft

Secondary changes

  • Set EuiInputPopover default offset to 2px to retain tighter association for inputs (effectively no visual change)
  • Change EuiTourStep to keep the arrow when beacon is present (effectively no visual change)
  • Change EuiTourStep story to use popover default instead of hardcoded value
  • Update positioning tests to reflect new default behavior

Why are we making this change?

Part of a larger effort to modernize our product UIs.

Screenshots

EuiPopover

(default 4px offset)
CleanShot 2025-10-07 at 16 31 23@2x

EuiInputPopover (visually unchanged)

(default 2px offset)
CleanShot 2025-10-07 at 16 30 43@2x

EuiContextMenu (in Popover; this is an existing storybook)

CleanShot 2025-10-07 at 16 33 11@2x

EuiTour (visually unchanged)

CleanShot 2025-10-07 at 16 36 42@2x

Impact to users

Minimal - visual only

QA

Remove or strikethrough items that do not apply to your PR.

Warning

Currently skipping a couple of tests to see if build succeeds. Need to look at these again.

General checklist

  • Browser QA
    • Checked in both light and dark modes
    • Checked in both MacOS and Windows high contrast modes
    • Checked in mobile
    • Checked in Chrome, Safari, Edge, and Firefox
    • Checked for accessibility including keyboard-only and screenreader modes
  • Docs site QA
  • Code quality checklist
  • Release checklist
    • A changelog entry exists and is marked appropriately.
    • If applicable, added the breaking change issue label (and filled out the breaking change checklist)
  • Designer checklist
    • If applicable, file an issue to update EUI's Figma library with any corresponding UI changes. (This is an internal repo, if you are external to Elastic, ask a maintainer to submit this request)

- Update EuiPopover default offset from 0 to 4px for better visual spacing
- Disable arrow by default (hasArrow: false) for cleaner popover appearance
- Fix EuiTourStep to use popover default instead of hardcoded 0
- Update positioning tests to reflect new default behavior
@ryankeairns ryankeairns force-pushed the rk/8931-popover-hide-arrow branch from 1ddf661 to 2c97900 Compare October 8, 2025 04:14
@elasticmachine
Copy link
Collaborator

πŸ’š Build Succeeded

History

@elasticmachine
Copy link
Collaborator

πŸ’” Build Failed

Failed CI Steps

History

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[EuiPopover] Remove arrow pointer

2 participants