React and SCSS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hello, Célestin Delahaye! 👋

  Good work on this challenge! 👍 Kudos for trying out Sass! 👏 I like the animation of the mobile menu icon! 😀

  A few things I'd like to suggest are,

  • Adding a max-width to the content of the page to prevent it from becoming too wide on extra-large screens.
  • Adding some anchor tags to either make the titles of the article cards or the article cards themselves links.
  • Taking another look at the layout of the article cards and making sure there's an even amount of space between them as much as possible (currently the space between the cards seems to vary between just right and a little too much when the size of the screen changes).

  Hope those tips help. 🙂

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Célestin Delahaye•145

  @ZeeeratulPosted over 3 years ago

  Hello ApplePieGiraffe ! Thank you so much for your feedback, I will definitely think about the max-width for my next project 👍 I agree with you: the layout for the articles isn't really good, I should add more space between them !

  Thanks again and Happy coding 😁

  1