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

Nav Drawer retains high z-index when position is relative #344

Closed
damyanpetev opened this issue May 4, 2022 · 0 comments · Fixed by #349
Closed

Nav Drawer retains high z-index when position is relative #344

damyanpetev opened this issue May 4, 2022 · 0 comments · Fixed by #349
Assignees

Comments

@damyanpetev
Copy link
Member

Description

When using a relative Nav Drawer (say in a combination with Nav Bar) the drawer still retains the z-index of 999 as if it was in overlay mode. Small oversight when introducing the overrides for relative position, but it's extremely visible in the Material theme as it clips the Nav Bar's elevation shadow:
image

Steps to reproduce

  1. Open the attached sample
  2. Observe the Nav bar's shadow

Result

The drawer is still using the high z-index and clipping other content, when it shouldn't.

Also I noticed the Nav Bar does have z-index: 4 on it's base part but that doesn't seem to work for some reason unless I move it to the :host. Same approach works for the drawer, so not sure what's up with that, so @simeonoff let me know if I should log that separately

Expected result

We should reset the z-index when in position relative

Attachments

wcNavBarAndDrawer.zip

@damyanpetev damyanpetev added bug Something isn't working 🆕 status: new The issue is new and will be reviewed when somebody picks it up. navbar nav-drawer labels May 4, 2022
@simeonoff simeonoff assigned SisIvanova and unassigned simeonoff May 5, 2022
@SisIvanova SisIvanova added 🛠️ status: in-development and removed 🆕 status: new The issue is new and will be reviewed when somebody picks it up. labels May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants