Responsive News Homepage

Community feedback

• 

  Adriano•38,390

  @AdrianoEscarabotePosted almost 2 years ago

  Hello Ahmad Aroyehun, how are you? Welcome to the community of front-end mentors! I truly loved your project's outcome, however I have some advice that I hope you'll find useful:

  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 solve this problem do this:

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

  If we see how the layout is behaving at higher resolutions, with the help of google dev tools, we will see that it is stretching a lot, to solve this we can use a max-width with the value we want the content to stop growing and to center use a margin: 0 auto;

  @media screen and (min-width: 35rem)
  .overall-container {
      max-width: 1440px;
      margin: 0 auto;
  }
  

  The remainder is excellent.

  I hope it's useful. 👍

  Marked as helpful

  0
• 

  Ahmad Aroyehun•220

  @ahmad-codPosted almost 2 years ago

  Thanks Adriano!

  0