Skip to content
Submitted about 2 years ago

Social proof

Pedro•130
@pdroaq
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

Community feedback

  • Lucas 👾•104,340
    @correlucasPosted about 2 years ago

    👾Hi Pedro, congrats on completing this challenge!

    I've just opened your live site and I can say that you did a great job putting everything together! There's some tips to improve your solution:

    1.You need to include the title for you PAGE. Do that inserting in the <head> the tag <title> → <title>Social Proof Comp - Front End Mentor</title>

    2.The html markup for this solution its built entirely with <div>and you can improve it using semantics tags to describe what its inside each block of content. You can replace the <div> that wraps each card with <article> you can wrap the paragraph with the quote with the tag <blockquote> this way you'll wrap each block of element with the best tag in this situation. Pay attention that <div> is only a block element without meaning.

    ✌️ I hope this helps you and happy coding!

    0
  • バレンタイン 😈•64,170
    @VCaramesPosted about 2 years ago

    Congrats @pdroaq on completing this challenge!

    You're content looks good, unfortunately, it isn't responsive.

    You'll want to add media queries to your CSS code and use *rem/em instead of px.

    Happy Coding!

    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