Skip to content
Submitted about 2 years ago

Responsive page using Flexbox and Grid

Doina•220
@Doileo
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 there, Frontendmentors! How are things working for you? Below is the project's solution. Because it's been a while since I've used Grid for layouts, I enjoyed it. Although I do not have any specific questions to ask, I would appreciate any feedback you may have. I am always open to learning things from a different perspective.

Have a fantastic day everyone, and Happy Coding!

Community feedback

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

    👾Hello Doina, Congratulations on completing this challenge!

    I saw your solution preview site and I think it's already really good. Here’s some tips for you to improve it:

    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.

    ✌️ I hope this helps you and happy coding!

    Marked as helpful
    0
    Doina•220
    @DoileoPosted about 2 years ago

    Hi @correlucas, thank you for taking the time to go over my solution and share valuable feedback. I appreciate it!

    I hope you have a great day!

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

    @Doileo You're welcome Doina! Keep posting good content =)

    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