Skip to content
Submitted 2 months ago

Testimonials Grid Section (mobile first using grid and flex)

AKdeBerg•160
@AKdeBerg
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?

I have implemented the desktop version using Grid and the mobile version using FlexBox. Using two types of layout methods boosts my confidence.

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

In desktop version, an item wasn't stretching all the way through the whole cell. Later I found a property called align-items: stretch which solves the problem.

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

Box shadow is still challenging for me to apply. I can see the two white cards has some shadow, it's hard for me to apply. Anybody help me how to have that kind of shadow?

Community feedback

  • Serhii Orlenko•345
    @grifanoPosted 2 months ago

    Great job on structuring the testimonial section using the <article> tags—it really improves the semantic clarity! Just a couple of suggestions to make it even better:

    • Accessibility: Consider adding <header> tags for the card titles and using role="list" and role="listitem" for improved accessibility.
    • CSS Grid: Named grid areas could make your code more readable and maintainable, rather than relying solely on nth-child.
    • SEO & Performance: Implementing structured data for the testimonials could boost SEO, and using lazy loading for images might improve page speed.

    Overall, it’s looking solid! Just a few tweaks, and it’ll be even more robust. 😊

    Marked as helpful
    1
    AKdeBerg•160
    @AKdeBergPosted 2 months ago

    great help! thank u so much @grifano

    0
  • Teodor Jenkler•3,740
    @TedJenklerPosted 2 months ago

    Hi @AKdeBerg,

    Nice project! I see you’ve put effort into accessibility, which is great. Here are a couple of tips for improvement:

    ARIA Labels: When using div elements instead of semantic HTML, consider adding descriptive class names to aria-label="". This will enhance accessibility and a quick fix—just copy the class name into the ARIA label.

    Font Weights: For an extra touch, you might experiment with font weights to get even closer to a pixel-perfect design. While not essential, it can help you achieve a near-perfect match with your screenshots.

    Hope these tips are helpful!

    Best, Teodor

    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