Skip to content
Submitted about 3 years ago

HTML CSS FLEXBOX GRID RWD

Olaf•230
@ziarnko
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social proof section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hey!!! This time I spend just a little bit more time on this project.Hope you will like it. Chcek code and pls pls for some advise about it. Thanks a lot for your time!!!

Community feedback

  • Olaf•230
    @ziarnkoPosted about 3 years ago

    Thank you mate.I going to update main.css soon.

    0
  • dewslyse•3,025
    @dewslysePosted about 3 years ago

    Hello Olaf! Congrats on your submission. Looking good, well done. A couple of things to note:

    • Replacing <div class="container"> with <main class="container"> will include a landmark element in your document and resolve all the accessibility issues.
    • You should set the max-width on your container to something less than 100% (say 60% or even 1110px) to prevent the content from spanning the full width of the page on larger devices.

    All the best with subsequent challenges

    0

Please log in to post a comment

Log in with GitHub

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