Design comparison
SolutionDesign
Solution retrospective
The thing I found to be the most difficult was figuring out how to import fonts.
Community feedback
- @danielmrz-devPosted 11 months ago
Hello @sandbrock!
Your project looks great!
I have a few tips for you to improve it:
- If you add a
box-shadow
to your card, it'll look even closer to the original design. Something like this is enough:
.card { box-shadow: 10px 10px 0 black; }
- I noticed that you used
margins
to place your card closer to the middle of the page. You can center an element by adding this to the body:
body { height: 100vh; display: flex; justify-content: center; align-items: center; }
By doing that, you won't need
margins
to move your card.- Use semantic tags. Your
div.card
can be replaced withmain.card
. The main title could (and should) be ah1
and so on. This can change a few things visually but it makes your HTML code more semantic and improves SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than that, you did a great job!
Marked as helpful0 - If you add a
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord