Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

responsive landing page

Olaphoxy 60

@olaphoxy

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Ken 935

@kenreibman

Posted

Great job! Just a suggestion to get rid of your accessibility issues:

Stray away from using heading tags to match the font size. Make sure you are increasing the heading tags from h1-h4, etc. Don't skip h2 and h3. I recommend changing your h4 headings to h2 and changing the font size of h2 to match the card.

Your h1 seems a little too small compared to the original design.

You should also wrap your content in a main tag. In your case it would be <main class="cover"> Make sure you do that for your attribution section as well, but make it a footer tag.

I hope this helps and make sure you generate a new report once you fix this!

Marked as helpful

0

Olaphoxy 60

@olaphoxy

Posted

@lmaoken Thanks a lot, I'll fix it.

0
Olaphoxy 60

@olaphoxy

Posted

@lmaoken Done1

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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