Skip to content
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.

Sign up for GitHub

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

Jump to bottom

EuiHeaderBreadcrumbs popover arrow #7577

Closed
tomitank opened this issue Mar 13, 2024 · 1 comment · Fixed by #7580
Closed

EuiHeaderBreadcrumbs popover arrow #7577

tomitank opened this issue Mar 13, 2024 · 1 comment · Fixed by #7580
Assignees
@cee-chen
Labels
bug

Comments

@tomitank
Copy link

tomitank commented Mar 13, 2024 •
edited by cee-chen
Loading

Repro link: https://eui.elastic.co/v93.3.0/storybook/?path=/story/euibreadcrumbs--playground&args=type:application;max:2

Describe the bug
EuiHeaderBreadcrumbs popover arrow separated from content div

Impact and severity
Why? in older version the svg icon was inside the breadcrumb content div with same background.

Environment and versions

  • EUI version: 93.3.0
  • React version: 18
  • Kibana version (if applicable):
  • Browser: all
  • Operating System: all

Screenshots
Screenshot 2024-03-13 at 15-01-51 Elastic UI
@tomitank tomitank added bug ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible labels Mar 13, 2024
@cee-chen cee-chen removed the ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible label Mar 13, 2024
@cee-chen cee-chen self-assigned this Mar 13, 2024
@cee-chen cee-chen mentioned this issue Mar 13, 2024
Merged
7 tasks
@cee-chen
Copy link
Contributor

Thanks a million for catching this @tomitank! 🙏 Fix coming in #7580

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

@cee-chen cee-chen

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[EuiBreadcrumb] Fix styling on application types with popovers + perf improvements cee-chen/eui
2 participants
@cee-chen @tomitank