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

Celine/expanding menu #20

Closed
wants to merge 11 commits into from
Closed

Celine/expanding menu #20

wants to merge 11 commits into from

Conversation

celinechoiii
Copy link
Collaborator

🍞 What's new in this PR

🥐 Description

  • added an expanding menu bar component that is to be used throughout the app
  • added pomegranate color to the global styles

🥖 Screenshots

Screenshot 2024-10-27 at 2 26 55 AM Screenshot 2024-10-27 at 2 27 20 AM

🥪 How to review

  • use ExpandingMenu instead of menu icon svg in the activeEvents/page.tsx file
  • play around with it! see how the transitions are
  • check mobile screen as well

🥧 Next steps

  • add routing to the nav bars
  • might need to create a layout component to ensure that menu bar only appears for certain pages

🥞 Relevant links

🥨 Online sources

🥯 Related PRs

CC: @celinechoiii

aidenm1 and others added 10 commits October 16, 2024 18:48
…se and styling (#16)

* [feat + style] user authentication + styling

* eslint style fixes

* typo in text for signup page

* forgot to capitalize the 'in' in 'Sign In'

* updated imports + links + added exclamation points

* updated imports + links + added exclamation points

---------

Co-authored-by: Rohin Juneja <[email protected]>
* onboarding context

* feat: more global colors
@celinechoiii celinechoiii requested a review from jxmoose October 28, 2024 03:23
@celinechoiii celinechoiii force-pushed the celine/expanding-menu branch from a211806 to 5cc2efa Compare October 28, 2024 04:58
@celinechoiii celinechoiii deleted the celine/expanding-menu branch November 23, 2024 00:29
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.

5 participants