codersforcauses / coexist

https://coexist-events.vercel.app
MIT License
6 stars 0 forks source link

I7 create navbar #25

Closed jasonkeo closed 4 months ago

jasonkeo commented 4 months ago

Change Summary

-made rough draft of navbar based on figma design => Home Page - Events View (Image Navbar) V1 -couldnt figure out how to implement reverse dropdown menu animation

Change Form

Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

Other Information

[Is there anything in particular in the review that I should be aware of?]

jasonkeo commented 4 months ago

i updated most of the styles to tailwind besides a few here and there either bc tailwind doesnt offer it or the preset is too different from what i want.

I got stuck on some other things tho. hopefully some front end demon can implement the dropdown menu animation cus idk what da fuq going on with that and theres another issues where it starts spazzing out when u go below 400px so that will prob be a problem for some phones screens.

to fix issues: -fix responsiveness (for under 400px width) -add dropdown menu animation

jasonkeo commented 4 months ago

Nicee 🫡

SetroZ commented 4 months ago

The dropdown from shadcn can't load font-sans for some reason...... It loads all other fonts though

dct0 commented 4 months ago

The dropdown from shadcn can't load font-sans for some reason...... It loads all other fonts though

It's because the dropdown uses a react portal that renders outside of the main div, which is where the font gets applied. See https://www.reddit.com/r/nextjs/comments/144bi0r/radix_ui_dropdownmenu_font_issue/ for a fix

SetroZ commented 4 months ago

can't find a way to get the type check to work. Leaving it to you guys ;) npm run typecheck works on my machine tho