Submitted 5 months ago
Testimonials Grid Layout built with HTML and CSS
@abubakar-sadiq001
Design comparison
SolutionDesign
Solution retrospective
What are you most proud of, and what would you do differently next time?
Every challenge I solve opens the door to new experiences and knowledge to flow😍
What challenges did you encounter, and how did you overcome them?I spent 30mins+ just trying to match the backgrounds with CSS color pallets. but finally, it helps me get my back covered.
What specific areas of your project would you like help with?box1 { background: hsl(263, 55%, 52%); } box2 { background: hsl(213, 20%, 28%); } box3 { background: #fff; } box4 { background: #1d1728; } box5 { background: #fff; }
Community feedback
- @YuliaLantzbergPosted 3 months ago
Hi. Looking nice. But it's better to use rem or em instead of px for better responsiveness. Also, I think there is a bit of confusion about fonts styling. According to the documentation :
- "Verified Graduate" has the same color as the person's name with 50% opacity
- Review paragraphs inside the quotations have the same color as well, but are at 70% opacity And I think, you set the 0.7 on name and no opacity on the quote text. And about the background colors, you have them defined in the style-guide.md file. Hope this is useful. Regards
Marked as helpful0
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