neven.cebic• 510
@redizork
Posted
You have no responsive design on this..
Marked as helpful
1
Yulia• 320
@YuliaLantzberg
Posted
@redizork You are right. It was a bug in my media query. Now fixed. Thank you so much for pointing out
1
Submitted
Testimonials component using grid and flex layouts and css variables
Yulia• 320
@YuliaLantzberg
Design comparison
Solution retrospective
What are you most proud of, and what would you do differently next time?
I started to feel more fluent about grid, flex and CSS in general.
Community feedback
Erick Felipa Pineda• 160
@Efelipa
Posted
Great solution Yulia, Both the design and the code are clean, superb. An efficient way to use grid could be to create a structure where only the cards are placed.
<section class="grid | other classes">
<article>
... Card header and stuff
</article class="card">
</section>
0
Yulia• 320
@YuliaLantzberg
Posted
@Efelipa Hi Erik. Thank you so much for your feedback. But I think I didn't really understand what you meant. I have a div with class 'cards' serving as a grid element container, so this container is actually the structure you are talking about. Or you meant something else and I've missed the point?
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