News homepage

Community feedback

• 

  Hassan Arafa•40

  @HassanArafa53Posted 11 months ago

  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

  ---

  

  Namrata Sinha•530

  @nmrtsnhPosted 11 months ago

  @HassanArafa53 Thank you for the feedback!

  0