Skip to content
Submitted almost 2 years ago

Responsive news homepage using css grid

Eyram•270
@EYRAM565
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

feedback on how to improve on the layout especially its responsiveness.

Community feedback

  • Happy Phiri•430
    @happy-phiriPosted almost 2 years ago

    Good solution here. One thing that could improve the layout could be adding a container class that you'll use to add some padding to the body. Another could be adding the hover state to the actual heading (h2, h3) rather than to the whole div, like so - .container1 h2:hover {}. This should add the hover states to the h2 in container1 div rather than to the whole div. Otherwise well done.

    Marked as helpful
    0
    Eyram•270
    @EYRAM565Posted almost 2 years ago

    @happy-phiri thanks for the feed back, i will work on it .

    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