Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive Page using CSS Grid

#webpack#accessibility
dannxvc 180

@dannxvc

Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hope you can give me some advices about the way I used CSS Grid! Thank you.

Community feedback

bunee 2,060

@buneeIsSlo

Posted

Hey! @dannxvc, looks like you've pasted the wrong link to your repo, be sure to fix it.

As far as your solution is concerned, it looks great and responds very well so good job!

Adding the alt attributes to your images should solve most of the accessibility issues. Hope this helps :)

Marked as helpful

1

dannxvc 180

@dannxvc

Posted

@buneeIsSlo ohh thank you so much! Right, accessibility is key. Please let me know if you can see my repository now, whenever you can, thank you! :)

1
bunee 2,060

@buneeIsSlo

Posted

@dannxvc Hi again, I took look at your code and it's quite clean and well maintained however, your markup can be made more semantic. For instance, the quotes on each card would be more appropriate inside a <blockquote> rather than the good ole <div>. You can learn more about it here

Apart from that everything looks great, keep it up!

Marked as helpful

1
dannxvc 180

@dannxvc

Posted

@buneeIsSlo Amazing! Thank you so much!!

1

Please log in to post a comment

Log in with GitHub
Discord logo

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