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.

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

[Bug]: Dialog in stacking context behind Header #3309

Open
3 tasks done
xXNickznXx opened this issue Dec 18, 2024 · 1 comment
Open
3 tasks done

[Bug]: Dialog in stacking context behind Header #3309

xXNickznXx opened this issue Dec 18, 2024 · 1 comment
Labels
type: bug 🐛 Something isn't working

Comments

@xXNickznXx
Copy link

xXNickznXx commented Dec 18, 2024

Preflight Checklist

Bug type

Accessibility

It affects the following packages

Components

Which version of Lyne Components are you using

2.0.1

Operating system

No response

Browser / Browser version

No response

Input mode

None

Additional settings

No response

What happened?

Dialogs with a lot of content get partially hidden behind the header if they are in a different stacking context.

I have noticed this on desktop with dialogs inside the sidebar of the map-container element: https://stackblitz.com/edit/on-desktop-dialog-in-map-container-behind-header?file=src%2Fmain.html,src%2Fstyles.css

I have also created a simple reproduction without the map-container: https://stackblitz.com/edit/dialog-in-stacking-context-behind-header?file=src%2Fmain.html,src%2Fstyles.css

Steps to reproduce the issue

map-container example:

  1. Open stackblitz
  2. Be on desktop (sidebar and map side by side)
  3. Click the "Open dialog" button
    Result: dialog partially behind header

without map-container example:

  1. Open stackblitz
  2. Click the "Open dialog (behind header)" button

Result: dialog partially behind header

  1. Click the "Open dialog (infront header)" button

Result: dialog is infront of header

Relevant log output

No response

@jeripeierSBB
Copy link
Contributor

Hi @xXNickznXx, thanks for the report. We can't improve this until we can use the popoverAPI (needs 0.5 to 1 year). But you could place the dialog outside your stacking context while keeping the button to trigger it inside.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug 🐛 Something isn't working
Projects
Development

No branches or pull requests

2 participants