Design comparison
SolutionDesign
Solution retrospective
Hope you could give me feedback. Thanks!
Community feedback
- @IEdiongPosted almost 3 years ago
Hello Khaled 👋, this is a well built component. Some suggestions:
-
The images are meant to have a colored border around it, refer to the design.
-
This is a personal suggestion: I feel you could play around with the layout at different screen sizes even though it wasn't given in the design. You could refer to my solution for this challenge to see how the layout changes at different screen sizes.
-
Also look into the accessibility issues raised here.
I hope this was helpful, Shalom!
Marked as helpful0 -
- @KhaledAhmedKassemPosted almost 3 years ago
@IEdiong Thanks for your feedback
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