Design comparison
Community feedback
- @correlucasPosted over 2 years ago
πΎHello Rosalia!
Congratulations for your new challenge solution!
I've inspected your solution's code and I've some tips for you:
1.Clean the overall html structure by removing unnecessary elements, for example, you're wrapping everything with the
<main>
tag so there's no need to use<section>
under that. You can also use the selector instead of adding a class to manage its content, doing that you've less lines code of written and more perfomance.2.The container is really good well done, but isn't fully responsive yet due a single property inside the .container class the
width
, so you can replace withmax-width
in order to fix this issue.The rest is all amazing, you've done the hardest part that was the double circles on the background.
Rosalia, I hope this was helpful and happy coding. Have a nice day.
Marked as helpful0
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