news-homepage-main

Community feedback

• 

  Adriano•38,120

  @AdrianoEscarabotePosted almost 2 years ago

  Hello Taofeeq Olatigbe, 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>

  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.

  nav,
  .container,
  .blog {
      max-width: 1440px;
  }
  

  The remainder is excellent.

  I hope it's useful. 👍

  0
• 

  Sarib•20

  @Sarib-SiddiquePosted almost 2 years ago

  Hello, Because you have to learn javascript to change menu SVG to closing SVG. By the way great work, You did well.

  0

  ---

  

  Taofeeq Olatigbe•50

  @TaofeeTechPosted almost 2 years ago

  @Sarib-Siddique thank you so much

  0
• 

  Sinisa Vukmirovic•1,690

  @SinisaVukmirovicPosted almost 2 years ago

  Hello!

  You need JavaScript code to change the menu icon SVG.

  Also, your "VIEW CODE" link leads to an empty repository.

  0

  ---

  

  Taofeeq Olatigbe•50

  @TaofeeTechPosted almost 2 years ago

  @SinisaVukmirovic oh okay thanks,

  0