Submitted over 3 years ago
Website using html, scss, BEM, mobile first workflow
@JuanDa15
Design comparison
SolutionDesign
Solution retrospective
open to feedback and recomendations
Community feedback
- @jerry-the-kidPosted over 3 years ago
At
.card__image img
You can addobject-fit: cover
, so your image have responsive.
In Your
Body
usemin-height : 100vh
instead ofheight: 100%
so in your mobile version, the card isn't hidden.
There are my advices and i think in your
.card__content
has so much space in bottom (desktop-version). Your solution is good anyway. Well done 🎉✨Marked as helpful1@JuanDa15Posted over 3 years ago@jerry-carry Thank's for your recommendations, I will apply them
1 - @palgrammingPosted over 3 years ago
⭐⭐⭐⭐⭐ Good job on this challenge no complaints. You should be happy with your results
Marked as helpful1
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