Submitted over 2 years ago
Responsive Landing page using Flexbox, CSS grids and SASS processor
@OGEmanuel
Design comparison
SolutionDesign
Solution retrospective
I couldn't seem to find a way to clip off the right side of the desktop image on the larger screen. Anybody got any tips?
Community feedback
- @Bayoumi-devPosted over 2 years ago
Hey Ogunmola, Congratulations on completing this challenge... You have some
accessibility issues
that need to fix.Document should have one main landmark
, Contain the main content in<main>
.
<main> <section class="section-info"> //... </section> <section class="section-features"> //... </section> <section class="section-partners"> //... </section> </main>
Heading levels should only increase by one
, Changeh6
toh3
.- For
SEO
, Only Useh1
Elements Once Per Page... More details Links must have discernible text
, Set the attributearia-label
to describe the link.
I hope this is helpful to you... Keep coding👍
Marked as helpful1@OGEmanuelPosted over 2 years ago@Bayoumi-dev yes, thank you so much. I appreciate it 💯. I'll make sure to fix up.
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