Loopstudio Landing Page using SASS, HTML and Vanilla JS

Community feedback

• 

  Albertoflj•170

  @AlbertofljPosted over 3 years ago

  Hey, ApplePieGiraffe! 👋

  Thank you a lot for your feedback! I will definitely use a button element in the future!😊

  And thank you for clarifying the utility classes, I will try to work on landing pages more often and look for recurring styles. 😄

  Happy coding to you too! 😁

  0
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hello there, Albertoflj! 👋

  Great job on this challenge! 👏 Your solution looks good and responds nicely and I like the transition that you added to the intro of the mobile menu! 😀

  The only very small thing that I suggest is to perhaps use a <button> element for the mobile menu toggle since, well, it's more of a button and doesn't take the user anywhere (like a link would). 😉

  As for utility classes, I don't use them all that much, but I suppose just keep an eye out for any values that you believe can be or will be reused across your page or site (perhaps things like margins around things, etc.) and then... create a utility classes for those. 😅 I often just go with styling elements more or less individually, but if you spot a recurring pattern in your styles it could be helpful! 🙂

  Keep coding (and happy coding, too)! 😁

  0