Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

News Home Page Using Grid

Ramel A. Leceraβ€’ 90

@RAMZY329

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


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

@sulemaan7070

Posted

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

@RAMZY329

Posted

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

1

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