-
Notifications
You must be signed in to change notification settings - Fork 3
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
Buyankhuu/new footer #27
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
||
import { FooterDiv, ContactDiv } from '../styles/components'; | ||
|
||
export default function Logo() { |
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.
change the function name, Logo gets reused across multiple pages rn. generally good practice is to name it after the page or component i.e. here it would be export default function Footer()
@@ -0,0 +1,17 @@ | |||
import { LocationDiv, Addie } from '../styles/components'; | |||
|
|||
export default function Logo(props: { |
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.
same thing here abt function names, name it something like Location()
display: flex; | ||
flex-direction: row; | ||
color: black; | ||
width: 90%; |
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.
NO PERCENTAGES (unless u are doing 100%)--stick w pixels unless told otherwise
Description
Created a popup using toasty for the logout button which will redirect the user to the login screen.
Footer for all the pages was created
New components:
Footer: the entirety of the footer
location: the small div with the text of each location that Shanti has
added the logo to the login screen and got rid of the yellow box
Screenshots
https://www.loom.com/share/58a7630a1d24461597a765d1afb769ac?sid=f2700b80-fcc4-4901-b9bb-6ff3e24f70b2
How to review
Order: Component/Location -> Component/Footer -> style/component -> Profile/page -> Profile/style
Testing:
go to localhost
go to each page and scroll to the bottom to see the footer
navigate to localhost/profile
press logout -> popout shows up -> send user to login page
Next steps
next step: see if design of footer changes/ popup
Relevant Links
Online sources
Related PRs
🛍 CC: @EthanAuyeung