Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Testimonial Page

James. 150

@Jimztech

Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


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

I had a firm grip of the application of css grid and flexbox in this project.

Community feedback

@TedJenkler

Posted

Hi @Jimztech,

Nice project! Here are a few suggestions for improvement:

Semantic HTML: Consider replacing <div> elements with more semantic HTML elements. For instance, use <main> instead of "main-content," and <article> or <section> instead of "cards." This change will improve both accessibility and SEO.

Heading Structure: Keep in mind that <h1> should be used only once per page to represent the main heading. For other sections, <h2> is preferable as the first heading. This helps maintain a clear and logical heading structure.

Keep up the great work!

Best, Teodor

Marked as helpful

0

James. 150

@Jimztech

Posted

Thanks @TedJenkler

1

@GhassanAlKaraan

Posted

wow cool work.

the gap between the cards should be the same horizontally and vertically.

if you're using grid, it could be:

gap: 2rem;

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

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