Skip to content
Submitted about 1 month ago

Responsive landing page using Grid

#pure-css
Eziefule Judith•110
@a2-112
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Testimonials grid section
  • HTML
  • CSS
2junior

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

Getting this project done i will definitely keep using Grid

What challenges did you encounter, and how did you overcome them?

to get it positioned this way but thanks to mentor Kevin powell i did it

What specific areas of your project would you like help with?

grid

Community feedback

  • Ilay Zeitlin•180
    @Ilay-IlayPosted about 1 month ago

    Hi! The structure is great and follows the original perfectly.

    The only thing I would recommend is not to use the same line height for all the text. It's a good practice in design to use smaller line heights for bigger fonts (the bigger the font the smaller the line height and the opposite for small font, the smaller the font is the bigger the line height should be). This is used because the big font is already big and easy to read. You can use chat GPT so it will create a good line height for each font size.

    Marked as helpful
    0
    Eziefule Judith•110
    @a2-112Posted about 1 month ago

    @Ilay-Ilay Thanks i appreciate will definitely do that

    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