Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Request path contains unescaped characters
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive news homepage using css grid

Eyram 270

@EYRAM565

Desktop design screenshot for the News homepage coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


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

Community feedback

@happy-phiri

Posted

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

@EYRAM565

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

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