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

fix: z-level of mobile navgation #335

Merged
merged 1 commit into from
Jan 8, 2022
Merged

fix: z-level of mobile navgation #335

merged 1 commit into from
Jan 8, 2022

Conversation

holtergram
Copy link
Contributor

This PR increases the z-index value of the mobile navigation and navigation button to z-50 to properly hide other elements like "Copy to clipboard" buttons when navigation is open.
First reported with Issue #334 by @aryaniyaps

This PR increases the Z-Index of the mobile navigation and navigation button to 50 to properly hide other elements like "Copy to clipboard" buttons when navigation is open.
Issue first reported here: #334
@vercel
Copy link

vercel bot commented Jan 8, 2022

@2malh is attempting to deploy a commit to the Strawberry GraphQL Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Jan 8, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/strawberry-graphql/strawberry-rocks/7DNGaLdMF8Mww3BxNh1fWghmc9TK
✅ Preview: https://strawberry-rocks-git-fork-2malh-patch-2-strawberry-graphql.vercel.app

Copy link
Member

@patrick91 patrick91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Thank you!

@patrick91 patrick91 merged commit 605b8ff into strawberry-graphql:main Jan 8, 2022
@holtergram holtergram deleted the patch-2 branch January 8, 2022 23:14
@aryaniyaps
Copy link

Thanks a lot for the PR @2malh !!😊

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.

None yet

3 participants