News homepage with CSS Grid and Sass

Community feedback

• 

  Adriano•42,890

  @AdrianoEscarabote

  Posted over 2 years ago

  Hi misty, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

  I noticed that you used a button in which case the best option would be an a, because in my head when a person clicks on a button written Read more, he is not confirming a form, or something like, it will be redirected to another page, to read more about!

  To resolve do this:

  <a href="/">Read more</a>

  I noticed that in higher resolutions the content is stretching a lot and this is harming the design of the project, to fix this in a simple way we can use a max-width.

  body {
      max-width: 1440px;
  }
  

  The rest is great!

  I hope it helps... 👍

  10

  Marked as helpful