Submitted over 3 years ago
First try at making responsive using CSS Grid - feedback appreciated!
@mattyoung7
Design comparison
SolutionDesign
Community feedback
- @Jos02378Posted over 3 years ago
Hey @mattyoung7, Great job on this solution!
Some feedback for you:
- Try to use the alt attribute on images because the alt text is shown if the image can't load on the website to prevent confusion to the user.
- You can try to follow a CSS naming convention called BEM. You can watch this video for more information Link to video.
I hope this helps, good luck!
Marked as helpful0@mattyoung7Posted over 3 years ago@Jos02378 Thanks for the feedback, really appreciate it!
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