News homepage (React | Sass)

Community feedback

• 

  Wendy•1,990

  @wendyhamelPosted about 1 month ago

  Great solution to this challenge!

  Your solution is extremely close to the screenshot and resizes nice on smaller screens. It does have a scrollbar on larger screens, when all the content is visible, so you might want to adjust the .margin-block: 90px on <main> to just margin top for example.

  You might want to look into more use of semantic HTML to reduce the meaningless div's for better accessibility. The .news-container and .populair-container could be <article>'s instead of a <div>'s.

  Last, a very small design difference: in the screenshot the text of the read more button is black, but in the design it is white.

  Happy coding!

  1

  ---

  

  P

  Marc Francis•730

  @marcfrancissPosted about 1 month ago

  @wendyhamel Thanks checking my solution for this challenge and for the feedback! 😊 Really appreciate your suggestion on using <article> to improve the semantic structure.

  0