Design comparison
Solution retrospective
edit: frontend mentor has an issue with the screenshot could you please check if you have it on my solution site too?
I wish you to give me your feedback to be better in the future 😊♥.
Community feedback
- @zineb-BouPosted about 3 years ago
Hi Mohammed, I noticed couple of things in your solution, the desktop layout doesn't show up to me, I think you chose the mobile first approach, then using media queries to set up the desktop, so check again the min-width. keep coding...
Marked as helpful1@MohammedHany1Posted about 3 years ago@zineb-Bou Thank you for your helpful reply Zineb! I have checked the min-width again and changed a couple of things. could you please check the site again to see if the problem is gone?
0@zineb-BouPosted about 3 years ago@MohammedHany1 Hey Mohammed, I just checked again your solution, the problem was fixed, Good job,
0 - @0rGaan1cPosted about 3 years ago
It looks good on mine as well, probably an issue with frontendmentor, maybe try submitting it again.
Marked as helpful1@MohammedHany1Posted about 3 years ago@0rGaan1c I have just submitted it again but the problem is still there. I just want to check if it happens on a specific screen size or something like that so I can fix the bug If it is there. but thanks for your help :)
0 - @BianGodePosted about 3 years ago
It looks good. You can improve the width and height of the grid items. Keep on going!
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