News homepage (SASS,SCSS,NPM,autoprefixer) Using Flexbox.

Community feedback

• 

  Adriano•38,290

  @AdrianoEscarabotePosted about 2 years ago

  Hi Sabbir Ahmed, 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 this:

  <a class="btn btn__read-more">Read more</a>

  I noticed that in higher resolutions the content is stretching a lot, and this is causing a negative impact on the page design, to solve this we can do this:

  body {
      max-width: 1440px;
      margin: 0 auto;
  }
  

  The rest is great!

  I hope it helps... 👍

  1

  ---

  

  P

  Sabbir Ahmed•700

  @sabbircoder07Posted about 2 years ago

  @AdrianoEscarabote Thank you very much for your wonderful feedback and comments. Thanks again.

  1

  

  Adriano•38,290

  @AdrianoEscarabotePosted about 2 years ago

  @sabbircoder07 happy coding!!

  1