-
Notifications
You must be signed in to change notification settings - Fork 2
Add HomeView + Navbar #6
base: master
Are you sure you want to change the base?
Conversation
client/src/pages/HomeView/styles.css
Outdated
} | ||
|
||
.title { | ||
padding-top: 100px; |
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.
We want to use rem
or %
rather than px
where possible to accommodate for different display sizes.
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.
Navbar is fixed height – so accounting for that by padding. (Honestly, will probably define a layout component in the future that accounts for that and is applied to all the pages)
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/hack4impact/closegap/hc86odt5a |
…closegap into ec/add-nav-bar
Status: 🚧
Description
• add Home page to the app (has pink background and "Welcome" text)
• add Navbar component to the app
• Navbar with Closegap logo and login/logout buttons (based on current Closegap teacher view)
• add Head component that renders the Navbar component
Todos
Screenshots