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

feature:edited the page-header to serve as a navbar for mobile screen #85

Merged

Conversation

Vixs101
Copy link
Contributor

@Vixs101 Vixs101 commented Oct 15, 2024

** Closes #2
Hi so i noticed there was already a page header component on mobile, so I just added the logo, made the nav fixed, and added a border to the bottom of the nav to separate it from other contents.

here are some screenshots:
image

Screenshot from 2024-10-15 12-38-47

@harshmangalam harshmangalam changed the base branch from main to dev October 15, 2024 13:23
@harshmangalam harshmangalam merged commit a151eea into harshmangalam:dev Oct 15, 2024
1 check passed
@harshmangalam
Copy link
Owner

Hi @Vixs101 ,

Thank you so much for contributing to qwik-x during Hacktoberfest! Your efforts are truly appreciated, and we’d love to give you kudos and a shout-out for your amazing work.

If you're comfortable, please feel free to share your social media handles like linkedin, twitter,etc... so we can tag you in our appreciation posts and spread the word about your contributions!

Looking forward to seeing you in the community. Thanks again for being awesome!

Best,
Harsh Mangalam
Maintainer qwik-x

@Vixs101
Copy link
Contributor Author

Vixs101 commented Oct 21, 2024

HI @harshmangalam it's an honour contributing. thanks for letting a newbie like me help out.
My x handle is @Vixs101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add navbar in mobile screen
2 participants