Skip to content
Submitted about 2 years ago

News homepage - CSS Grid, Animations, JS, Mobile-first, BEM

Ben•160
@blue-crona
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...News homepage
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

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

  • polukarp•190
    @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 and margin: 0 auto to make sure your website looks good on ultrawide monitors.

    Let me know if you have any questions.

    Marked as helpful
    1
    Ben•160
    @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 GitHub

Join 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