Solution to Testimonial Grid challenge using HTML, SASS and CSS Grid
Design comparison
Community feedback
- @isprutfromuaPosted over 2 years ago
Hi there. You did a good job 😎
keep improving your programming skills🛠️
your solution looks great, however, if you want to improve it, you can follow these steps:
✅ quote symbol is broken. You can set it with pseudoelements and background-image
✅ your layout isn't responsive. You could fix it with media queries
✅ Missing unique Alt Text on Images
✅ Use relative units for font size, such as ems or rems. While modern browsers can smoothly zoom pixel-based layouts, sizing type in relative units ensures an entire layout can be scaled up or down by simply updating the font-size of the body element.
✅ pay attention to the automatic report on your solution. you need to fix html and a11y errors
I hope my feedback will be helpful. You can mark it as useful if so 👍
Good luck and fun coding 🤝⌨️
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