Design comparison
Community feedback
- @VCaramesPosted about 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
-
The
header
element needs to be outside of themain
element. -
The "Build The Community Your Fans Will Love" is the most important heading in you site, so it should be wrapped in an
h1
heading. -
Along with the blank
alt tag
, you also want to include thearia-hidden=“true”
to your Illustrations to fully remove them from assistive technology. -
"Grow Together", "Flowing Conversations" and "Your Users" should be wrapped in
h2
headings. -
The
footer
“phone” and “address” should be wrapped inAnchor Tags
, so users can easily click on them and have the appropriate app open for them. -
The “social media icons” are not decorative, they need to have an
Alt Tag
with a description. -
The
footer
links need to be wrapped inside anav
element. -
Implement a Mobile First approach 📱 > 🖥
With mobile devices being the predominant way that people view websites/content. It is more crucial than ever to ensure that your website/content looks presentable on all mobile devices. To achieve this, you start building your website/content for smaller screen first and then adjust your content for larger screens.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 🍂🦃
0 -
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