Skip to content
Submitted about 2 years ago

Social Proof Section

Adam•210
@AdamMintaj
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

I started paying the due attention to the semantic HTML tags only recently, so I'd be grateful if someone looked at my code and its structure and gave me some hints. Hit me with your harshest criticism!

Community feedback

  • Adriano•38,080
    @AdrianoEscarabotePosted about 2 years ago

    Hi @AdamMintaj, how are you?

    I really liked the result of your project, but I have some tips that I think you will like:

    I looked at your layout at higher resolutions and saw that it is growing a lot, to fix this we can do the following:

    body { max-width: 1440px; margin: 0 auto; }

    This way the layout will have a maximum width, preventing the content from stretching.

    the rest is great! Hope it helps... 👍

    0
    Adam•210
    @AdamMintajPosted about 2 years ago

    Hi @AdrianoEscarabote, many thanks for looking at my solution! You're the real MVP :) Thank you for your suggestion, I just work on a regular laptop and sometimes still forget that there are bigger screens too

    1

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