Design comparison
Solution retrospective
How do i make it mobile accesible? Also i think that styles are a lil bit off.
Community feedback
- @Durban86Posted over 1 year ago
Ok an error I found was in your HTML
Line 13: change the img id to "product-img"
and then remove "#product-img-desktop" from Line 125 of your CSS
Doing so will at least make every element visible for mobile, it doesn't look as nice as your desktop view which is very nice. I think you might have an easier go of this using flexbox instead of grid. Best of luck, well done on the desktop layout.
Marked as helpful0@WeberowskyPosted over 1 year agoIll fix the css today, but I don't know how to make the mobile view. I'll try to remake the css to support mobile but I dont know if it will work @Durban86
0@Durban86Posted over 1 year ago@Weberowsky I always build mobile first, then adjust for the desktop. Mobile for this project should be one column with a 375px width. I really think starting over with a mobile first process and with flex box will help you a lot.
Here’s my code to use for reference
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