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

Report

0

Accessibility Issues

0

HTML Issues

View Report

Community Feedback

Ken 455

@lmaoken

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

Give some feedback to @olaphoxy about their solution...

Slack logo

Join our Slack community

Join over 80,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!