Design comparison
SolutionDesign
Solution retrospective
What are you most proud of, and what would you do differently next time?
- Changed all px to rem
- Changed vars naming
- Pixel perfect
no difficulties
What specific areas of your project would you like help with?easy challenge
Community feedback
- @danielmrz-devPosted 10 months ago
Hello @ankkiel!
Your project looks great!
I noticed that you used 4 different containers for the main content.
š Unless you have a very specific reason, you can use just the tag
<main>
, since the main content goes inside of it.š The tag
<section>
, for example, is completely unnecessary. It would make much more sense if the card was part of a bigger website (in certainly would in real world), but here it is all we have on the screen.This will help you to reduce your code lines, leaving only what's necessary.
I hope it helps!
Other than that, great job!
Marked as helpful1
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