News homepage - CSS Grid, Animations, JS, Mobile-first, BEM
Design comparison
Solution retrospective
Hello, this is my attempt at the news homepage challenge.
I'd appreciate any feedback or suggestions that you might have regarding this solution. 😊
Community feedback
- @polukarpPosted about 2 years ago
Awesome solution, but I found that on mobile view your .info-grid-item-img aren't responsive. So what I suggest you do about it is use
object-fit: contain
on it and that's gonna solve this issue.Also I'm not sure about the padding here, I'd just use
max-width:1440px
andmargin: 0 auto
to make sure your website looks good on ultrawide monitors.Let me know if you have any questions.
Marked as helpful1@blue-cronaPosted about 2 years ago@polukarp I'm excited to try your suggestions.
Thank you for the 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