Design comparison
SolutionDesign
Solution retrospective
Btw i used the file i downloaded instead of creating a new one.Because i felt lazy to copy svgs, imgs etc.
Community feedback
- @Zy8712Posted 12 months ago
Your site looks pretty good. The only things I'd add/modify is a
<main>
tag to wrap around you page contents (for accesibility purposes) and using<h1>
instead of<h2>
. This is because headers are suppose to be used going from<h1>
to<h6>
without skipping any. All these headers come with default font sizes that can be changed using thefont-size
property in css.Hope you find this feedback useful 👍
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