Skip to content
Submitted almost 2 years ago

News Home Page Using Grid

Ramel A. Lecera•90
@RAMZY329
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

I find difficulty in the lay out. I am not sure about making the body a little bit darker when the button is clicked to show the menu. What are the best practice in setting the width for mobile screen?

Community feedback

  • S MD suleman•3,530
    @sulemaan7070Posted almost 2 years ago

    hey Ramel A. Lecera 😄, congratulations on completing the challenge. here are a few tips you can use and make your site look better.

    1. The hero img on the desktop version seems cropped you can probably fix that.
    2. on the mobile version the mobile-menu is not occupying the full screen.
    3. I saw you have used flex on the body and centered the whole thing. that is not a problem but the empty space around the layout is not looking good. you can increase the size of the divs inside the body. everything else seems good. keep up the good work. happy coding🔥🔥💯🎉
    0
    Ramel A. Lecera•90
    @RAMZY329Posted almost 2 years ago

    @sulemaan7070 thank you for the comment brother.. happy coding too.

    1

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