Testimonials grid section using CSS Grid and some flexbox
Design comparison
Solution retrospective
Feedback welcome!
Community feedback
- @correlucasPosted about 2 years ago
👾Hello again Dan, congratulations for your new solution!
Nice solution and nice code! I can see that you paid a lot of attention to your code/design. If you don’t mind I’ve some tips for you:
1.The only cards that has the profile photo purple borders are the
black
and thepurple
cards.2.You did a great work using semantic tags for the html markup, one block you can improve is the paragraph containing the quote text you can replace the
<p>
with<blockquote>
that is the exact markup that describe this kind of content.3.Use a CSS reset to avoid all the problems you can have with the default CSS setup, removing all margins, making the images easier to work, see the article below where you can copy and paste this css code cheatsheet: https://piccalil.li/blog/a-modern-css-reset/
✌️ I hope this helps you and happy coding!
Marked as helpful1@igafshellPosted about 2 years ago@correlucas Thanks for the feedback, I appreciate it very much
0
Please log in to post a comment
Log in with GitHubJoin 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