Skip to content
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

Open
wants to merge 29 commits into
base: master
Choose a base branch
from

Conversation

Nahnahke
Copy link

No description provided.

Copy link

@jonsjak jonsjak left a 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)-->
Copy link

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 />
</>
);
}
Copy link

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 = () => {
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants