Skip to content
This repository has been archived by the owner on Jan 24, 2025. It is now read-only.

V2: Menu icon in mobile view gets covered by other absolute components ☰ #1112

Closed
Dangoo opened this issue Sep 13, 2019 · 0 comments · Fixed by #1144
Closed

V2: Menu icon in mobile view gets covered by other absolute components ☰ #1112

Dangoo opened this issue Sep 13, 2019 · 0 comments · Fixed by #1144

Comments

@Dangoo
Copy link
Contributor

Dangoo commented Sep 13, 2019

The three bar menu icon (☰) in mobile view of gatsby-theme-docz has an initial z-index value and thus can be covered easily/accidentally by any other following absolute/fixed positioned sibling.

menu

This can be guarded by using an (arbitrary) higher index - even though any developer could choose a higher value of course.
Alternatively, moving the button out of the main container (e.g. to the header) can bring it into another stacking context so it can never be overlaid by children of main.

What do you think?

rakannimer pushed a commit that referenced this issue Oct 1, 2019
* feat: move menu icon to header

* feat: tweaks for chevron
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants