-
Notifications
You must be signed in to change notification settings - Fork 6
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
Component Reorganization #240
Conversation
each search bar is now it's own file
fix back button imports
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall, since you're doing this, we should make the switch over to using Club rather than Org now. Other than that, minor import nitpicks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rename this component
This pr reorganizes the contents of the Components folder so that they are better organized and grouped by the domain they relate to (Header, Directory, etc.)
Some components have been moved into new folders within the components folder, while others have been moved into the app folder. this determination of where things ended up is dependent on where and how the components are consumed.
also
ContactIcons
was moved to the icons folder.The searchBar has been extensively broken down in order to try and seperate the debounce logic from the styling itself, this should make it easier to reuse the component in places where the debounce & autocomplete behavior isn't wanted. Other changes to the searchBar will come in a later pr.