Design comparison
SolutionDesign
Solution retrospective
Feedback welcome :)
Community feedback
- @denieldenPosted over 2 years ago
Hi Renzo, great work!
But
section
go insidemain
tag not the other way around and instead of usingpx
try to use relative units of measurement -> read hereOverall you did well π Hope this help and happy coding!
Marked as helpful0 - @Akhlak-Hossain-JimPosted over 2 years ago
Great work Ranzo. Her is my recomandations to solve some of your issues.
- try wraping
<div class="attribution">Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. Coded by <a href="#">Renzo Alvarez</a>.</div>
this in a<footer>
tag. - don't use
<section><main>...</main></section>
instead use<main><section>...</section></main>
.
your other html css code looks good to me so try learning sass and frameworks to move forward.
Happy coding :)
Marked as helpful0@ralvarezrenzoPosted over 2 years ago@Akhlak-Hossain-Jim Thank you very much for letting me know, now I will correct the errors of the footer > attribution and section > main, greetings.
0 - try wraping
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