Skip to content
Submitted about 1 month ago

Testimonials Grid Section | Next.js, TypeScript, Tailwind Css

tloyan•275
@tloyan
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 could further separate the different parts of the code and rethink how to handle the variations between each, while using the same component with dynamic data. Definitely on my refactor list!

Community feedback

  • Amit Dhakal•530
    @herojk64Posted 27 days ago

    Well good structure but since you're using Figma I think it should be more accurate according to the design all the spacing and such.

    0
    tloyan•275
    @tloyanPosted 25 days ago

    @herojk64

    Yes, you’re right; there are some differences I’m not too happy with either. That’s why I’m planning to refactor this challenge and all the others I’ve done. Figma definitely helps a lot, but it doesn’t eliminate all the work! There are also many other areas I could improve. :)

    0
    Amit Dhakal•530
    @herojk64Posted 25 days ago

    @tloyan Yes true it doesn't eliminate all the work great job. but I was pointing toward your box sizes and all. Typography with line-heights is fine if you want to ignore it doesn't matter if it looks good. but in your design the flaw was the box sizing nothing else.

    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