News homepage solution

Community feedback

• 

  Selvy Atef•200

  @Selvy2004Posted 7 months ago

  Hi!, Good job:) But I have some notes for you:

  1. the background-color shouldn't be white, it should be off white hsl(36, 100%, 99%);

  2. h1's font-size, should be bigger, and its font-weight too should be bigger.

  3. In your HTML code, it will be a good thing to put all your page's content inside a <main></main> element as a container, after this give it in your css code {margin: 0 auto}, instead of 0 put your normal margin top and bottom, then the page will nicely centered.

  4. At the mobile (smaller screen) once you open the website, the navigation certainly open, it should open just when you click on the open icon.

  Good luck:)

  Marked as helpful

  0

  ---

  

  Nadir Bousalah•820

  @Medido1Posted 7 months ago

  I alredy have the page content in <main> but other than that i made the changes you suggested and it looks better. thank you so much :)

  1