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(frontend): fix Navigation component z-index for small devices #638

Merged
merged 2 commits into from
Oct 22, 2023

Conversation

Bitnagar
Copy link
Contributor

This PR fixes #618

Changes the frontend/src/Navigation.tsx component's z-index.

This was needed because the content that should be below the header was flowing over it in small screens. See below:

image

@vercel
Copy link

vercel bot commented Oct 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 22, 2023 3:24pm
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 22, 2023 3:24pm

@vercel
Copy link

vercel bot commented Oct 18, 2023

@Bitnagar is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@Bitnagar Bitnagar changed the title Fix(frontend): fix Navigation component z-index in for small devices Fix(frontend): fix Navigation component z-index for small devices Oct 18, 2023
@Bitnagar
Copy link
Contributor Author

Hi @tenokami . Hope you're well. I understand you might be busy. I was just wondering when can i expect this PR to get merged? Appreciate your time and efforts. 🙂

@tenokami
Copy link
Collaborator

@Bitnagar, @dartpain Will review and merge it as soon as he can. Thank you for your patience

@dartpain dartpain merged commit 81b16aa into arc53:main Oct 22, 2023
@Bitnagar Bitnagar deleted the fix(frontend)--fix-header-z-index branch October 22, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 Bug Report: Header is broken in mobile viewports.
3 participants