-
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
Move search bar to navbar #48
Move search bar to navbar #48
Conversation
@DylanHojnoski is attempting to deploy a commit to the Max Isom's projects Team on Vercel. A member of the Team first needs to authorize it. |
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.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #48 +/- ##
==========================================
- Coverage 33.29% 31.84% -1.45%
==========================================
Files 105 105
Lines 2481 2490 +9
Branches 406 410 +4
==========================================
- Hits 826 793 -33
- Misses 1590 1640 +50
+ Partials 65 57 -8 ☔ View full report in Codecov by Sentry. |
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.
pushed a small change to avoid layout shift between pages, looks good now 👍
This moves the search bar into the navbar to save on vertical space. It also works properly on smaller screens.