cpinitiative / usaco-guide

A free collection of curated, high-quality resources to take you from Bronze to Platinum and beyond.
https://usaco.guide
Other
1.62k stars 495 forks source link

Change Navbar Design #341

Closed thecodingwizard closed 3 years ago

thecodingwizard commented 4 years ago

Our current navbar can get quite cluttered: image

We might want to change some stuff (specifically changing the search bar to become a button that then opens into a full-width search?)

MrinallU commented 4 years ago

@thecodingwizard I can do this.

MrinallU commented 4 years ago

@thecodingwizard So other than changing the search bar what else would you need to me to get done?

thecodingwizard commented 4 years ago

Stuff here: https://github.com/cpinitiative/usaco-guide/labels/website (especially the Blog :P)

And also feel free to help with content issues as well.

Thanks! :)

MrinallU commented 4 years ago

@thecodingwizard Essentially what I had in mind was a component like the problem search, with a route in the navbar containing the new search component.

MrinallU commented 4 years ago

@thecodingwizard Can you assign me to this so I don't forget?

MrinallU commented 3 years ago

@thecodingwizard Just making sure a full width search just means a page like the problems search but with the modules

thecodingwizard commented 3 years ago

no... I was thinking of a search icon button that would expand to become full width when clicked (or something similar). Or, alternatively, when clicking the search icon a full-page popup for search appears; that is probably fine too. I just want the ability for users to search without having to leave their page.

MrinallU commented 3 years ago

Ohhhh, I got you!

Hanaffi commented 3 years ago

@thecodingwizard Something like this: https://www.youtube.com/watch?v=2yfLpNALc1g ?

thecodingwizard commented 3 years ago

actually, why don't we just make the search bar into a search button, and then when they click the search button a modal pops up? something like https://tailwindcss.com/

Would this be better?

Hanaffi commented 3 years ago

@thecodingwizard Yeah, Thats even better. I can finish it quickly. May you assign it to me?

thecodingwizard commented 3 years ago

alright, it's assigned :) thanks!

MrinallU commented 3 years ago

@thecodingwizard I'm almost done with this but it's fine if hanaffi wants to try.

thecodingwizard commented 3 years ago

I don't want to overload you since you're already working on numerous other issues, so if it's alright with you I think it's best if Hanaffi takes this one

MrinallU commented 3 years ago

@thecodingwizard That's true the BenQ repo browsing is kinda hard so I think he can take care of the easier ui issues way faster than me.

Hanaffi commented 3 years ago

@thecodingwizard Can you tell me where is that home page located? Im just not very prof with React&Gatsby so I cant find it :(

thecodingwizard commented 3 years ago

The navigation bar code is here: https://github.com/cpinitiative/usaco-guide/blob/master/src/components/TopNavigationBar/TopNavigationBar.tsx

(Feel free to refactor, it seems quite messy oops XD)

Hanaffi commented 3 years ago

@thecodingwizard Wow , Aside from the issue , Can you tell me how to approach complex stuff in react? I feel so beginner looking at your codebase , LOL. I built moviesapp and to-do list :(

MrinallU commented 3 years ago

@Hanaffi Try not to get overwhelmed, it seems big at first, but once you really take the time to read it most of the code becomes pretty trivial.

MrinallU commented 3 years ago

@Hanaffi What exactly are you having trouble understanding

Hanaffi commented 3 years ago

@MrinallU Well, I cant actually find the code needed to be edited in that file :(

MrinallU commented 3 years ago

ohhh

Hanaffi commented 3 years ago

Oh, just found it

MrinallU commented 3 years ago

@Hanaffi https://github.com/cpinitiative/usaco-guide/blob/master/src/components/TopNavigationBar/TopNavigationBar.tsx

Hanaffi commented 3 years ago

@MrinallU I cant understand the classes names. Is that a framework or what?

MrinallU commented 3 years ago

ummm no

MrinallU commented 3 years ago

If you don't understand the basic ideas you should check the React documentation: https://reactjs.org/docs/getting-started.html

Hanaffi commented 3 years ago

@thecodingwizard May I use material UI?

thecodingwizard commented 3 years ago

err probably not; that would add a large dependency to our project + we are using tailwind css already

Hanaffi commented 3 years ago

@thecodingwizard Can you tell me some hint on how to implement this popup using tailwind??

HarryWangATX commented 3 years ago

@Hanaffi https://tailwindui.com/components/application-ui/overlays/modals tailwind ui components has some good examples of their usage.

Hanaffi commented 3 years ago

@HarryWangATX Thanks.

Can someone tell me how to fix this issue? I mean , the search results goes out of the modal ! Untitled

bqi343 commented 3 years ago

looks like this is done?