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

Create Navbar Component #56

Closed
Aokijiop opened this issue Feb 5, 2024 · 0 comments · Fixed by #63
Closed

Create Navbar Component #56

Aokijiop opened this issue Feb 5, 2024 · 0 comments · Fixed by #63
Assignees

Comments

@Aokijiop
Copy link
Collaborator

Aokijiop commented Feb 5, 2024

Description

  • Create a new component called Navbar which matches the hi-fi design
    • Style it so that it has its position property set to "fixed"
    • The navbar should contain the 3 buttons as specified in the wireframes with the appropriate icons
      • To get the icons, click on the icons on Figma and export them as downloadables, which you can then place in our assets folder
        • For now, since the flow of the buttons hasn't been confirmed, you can just leave them functionless
    • In app.jsx create a wrapper for Navbar
      • Use the Outlet built-in provided by react-router-dom
      • Should look something like this:
        • image
      • Create a new Route in App.jsx and wrap our Playground page and DummyEvents page inside it
      • Should look something like this:
        • image
        • Except the routes would be for Playground and DummyEvents
      • After wrapping these pages with the wrapper, the Navbar component should appear in both pages

Acceptance Criteria

  • The Navbar component has been created and matches the specified design
  • In App.jsx, a wrapper for Navbar has been created and correct renders Navbar inside the Playground and DummyEvents page

Resources

@KatyH820 KatyH820 mentioned this issue Feb 9, 2024
3 tasks
@KatyH820 KatyH820 linked a pull request Feb 9, 2024 that will close this issue
3 tasks
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 a pull request may close this issue.

3 participants