-
Notifications
You must be signed in to change notification settings - Fork 57
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
Project Design Handover - Hannah Ek #41
base: master
Are you sure you want to change the base?
Conversation
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.
Since I don't have the design, it's hard to know exactly what it's supposed to look like but everything looks nice and functioning in my opinion. Good use and reuse of styled components.
There are some minor issues with the image caroussels, they act differently when spinning and some of the images gets out of line. .At first I didn't get that the images move because there is no arrow or other indicator of it, but I guess that's how the design is supposed to look.
Overall a very good job! Good looking design and well executed in code. Kudos!
<head> | ||
<meta charset="utf-8" /> | ||
<meta name="viewport" content="width=device-width, initial-scale=1" /> | ||
<link href="https://fonts.googleapis.com/css2?family=Libre+Baskerville:wght@700&display=swap" rel="stylesheet"> | ||
<link href="https://fonts.googleapis.com/css2?family=Abril+Fatface&display=swap" rel="stylesheet"> | ||
<meta property="og:title" content="Project Design Handover"> <!-- OG (open graph) tags are snippets of code that provide meta data about a web page, mainly to social media platforms (ex facebook, linkedin, instagram)--> |
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.
Nice to add metatags
<Footer /> | ||
</> | ||
); | ||
} |
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.
Nice App.js. Well structured and easy to follow!
@@ -0,0 +1,12 @@ | |||
import React from 'react'; | |||
|
|||
const PopUpModal = () => { |
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.
What is this part? Can't find any popup.
No description provided.