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

frontend: Add Navbar #40

Closed
wants to merge 1 commit into from

Conversation

criticic
Copy link
Contributor

@criticic criticic commented Jun 1, 2024

This adds a UNO themed navbar to the frontend. The navbar is responsive and collapses into a hamburger menu on smaller screens.

It can have multiple links, which can be defined on use.

Fixes #19

Description

Adds a Navigation Bar, (and removes some unnecessary css previously present, only using tailwind)

Motivation and Context

The design uses the UNO Red Color, and the logo is styled like the UNO Logo.

Checklist

  • I have tested these changes locally.
  • I have reviewed the code and ensured it follows the project's coding guidelines.
  • I have updated the documentation, if necessary.
  • I have assigned reviewers to this pull request.

Screenshots (if applicable)

image
image

@shivansh-bhatnagar18
Copy link
Owner

Thanks @criticic for the Pull Request and @AadityanshuSingh for reviewing it. The navbar needs more refinement but seems good enough to be merged. Just fix the Linting issues and we are good to go

@shivansh-bhatnagar18 shivansh-bhatnagar18 mentioned this pull request Jun 6, 2024
4 tasks
This adds a UNO themed navbar to the frontend. The navbar is responsive
and collapses into a hamburger menu on smaller screens.

It can have multiple links, which can be defined on use.

Fixes shivansh-bhatnagar18#19

Signed-off-by: Sagnik Mandal <[email protected]>
Copy link

vercel bot commented Jun 6, 2024

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

Name Status Preview Comments Updated (UTC)
multiplayer-uno ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 6, 2024 1:01pm

@criticic
Copy link
Contributor Author

criticic commented Jun 6, 2024

@shivansh-bhatnagar18 i have rebased the PR, kindly take a look now

@kuv2707
Copy link
Collaborator

kuv2707 commented Jun 7, 2024

Looks like it would need to be worked on again in accordance with the new design.

@shivansh-bhatnagar18
Copy link
Owner

@criticic Thanks for the PR but I think we are working on the new design so I am closing this PR for now

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.

Design a Navbar component
4 participants