news-homepage

Community feedback

• 

  P

  Yulia•720

  @YuliaLantzbergPosted 3 months ago

  Hi. Very good solution, way better than mine. I can hardly give helpful feedback here. The only thing is, maybe to wrap the list items in nav into a tag as if it would be a real site, then they'd be hyperlinks. There is also no hover effect on those links. Other than that, code looks so clean and works perfectly.

  Marked as helpful

  0
• 

  Hendrixx•430

  @BeeAlmightyPosted 3 months ago

  Hi Weverson Godinho, Great attempt at the design given by Frontend mentor, i love how bold your solution is and how neat your code is. However, i noticed some things that could be improved on:

  • Avoid the use of divs and span and use more semantic HTML like nav, main, section, article as they help with web accessibility, screen readers and search engine optimization.
  • Input CSS transition whenever you had a hover state like:

   transition: background-color 0.2s ease-in-out
  

  on the parent element as it helps with smooth transitions to the pseudo state.

  Marked as helpful

  0