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 homepage

@nmrtsnh

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


Hello everyone,

I am Namrata and this is my solution for the News homepage challenge.

If you have any suggestions to improve my code, please feel free to share!

Thank you 🙏

Community feedback

@HassanArafa53

Posted

Great job!, this is actually one of the few solutions that i really liked and enjoyed looking it, just to make it perfect!, please use Margin instead of Padding on the header element as Padding is not meant to be used like this, to understand it better please look it up or watch YouTube videos on it. if you try to make the page smaller and bigger with the browser you'll notice the nav-bar and links moving around because you've used Padding which is totally wrong in this case. one more thing give the 3 pictures in the bottom of the page max-height of 130px. Again great job keep going ❤️.. one more thing at 1206 pixels the page goes into mobile version but the hamburger menu doesn't show up.

1

@nmrtsnh

Posted

@HassanArafa53 Thank you for the feedback!

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