Skip to content
Submitted almost 4 years ago

Testimonials grid section using CSS Grid

Karim•810
@Karimsamir112
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

hey , Is this good ?? and what is the mistakes I did ?

I care so much about feedback because it let me improving myself

Thanks in advanceز

Community feedback

  • Aayush Sood•1,175
    @soodaayushPosted almost 4 years ago

    your website is decent but there are some things I suggest you improve on.

    1. Fix all of the HTML and Accessibility issues. If you press view report, you will see all the issues in your code.

    2. You should probably use the pictures that they have given you in the files.

    3. The boxes have barely any spacing so I suggest adding padding to those boxes.

    4. The alignment of the boxes is kind of broken so I suggest making all heights of the boxes equal.

    5. There are also some weird line breaks in all of the paragraphs so I suggest adding more spaces to the boxes.

    6. Your content isn't centered so I suggest using CSS Flexbox to center the content.

    That's all I have to say!

    Happy Coding!

    0
    Karim•810
    @Karimsamir112Posted almost 4 years ago

    @WarBaddy okay thanks you so much to help me improve myself

    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