Skip to content
Submitted about 2 years ago

Social Proof Section

Ohida Elizabeth•320
@Lizzietrust
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,400
    @correlucasPosted about 2 years ago

    👾Hello @Lizzietrust, Congratulations on completing this challenge!

    Great code and great solution! I’ve few suggestions for you that you can consider adding to your code:

    1.You did a really good work here putting everything together, something you can improve its your code html markup and semantics. 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.

    This article from Freecodecamp explains the main HTML semantic TAGS: https://www.freecodecamp.org/news/semantic-html5-elements/

    2.Add the website favicon inserting the svg image inside the <head>. <link rel="icon" type="image/x-icon" href="./images/favicon-32x32.png">

    3.Add background-repeat: no-repeat to avoid the background image repeating multiple times.

    ✌️ I hope this helps you and happy coding!

    Marked as helpful
    0
    Ohida Elizabeth•320
    @LizzietrustPosted about 2 years ago

    @correlucas Thanks for the helpful comment, I'll apply that from now.

    0
    Ohida Elizabeth•320
    @LizzietrustPosted about 2 years ago

    @correlucas Thanks for the helpful comment, I'll apply that from now.

    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