Pure scss and practicing a lot more extensible/reusable scss component

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hello, once again, Raymart Pamplona! 👋

  Good job on this challenge! 👏 Your solution looks good and responds very nicely (as usual)! 😀 I like the hover state of the navigation links in the footer of the page! 👍

  I don't really see any issues with your solution—the only minor thing I suggest is to probably add object-fit: cover to the image in the hero section of the page in the mobile layout (since it is a bit stretched when the layout first changes from desktop to mobile). 😉

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  @ApplePieGiraffe Hey, thank you for reviewing and to be honest, if I were to add that property, that is my first time using it and knowing about it :>. I tested it and it's awesome so thank you very much APG and glad that you're back in here again for reviews^^

  1

  

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  @pikamart

  Haha, sure thing! Glad you learned something, too! 👍

  Yeah, I was quite busy last week so I took a short break from giving feedback. 😅

  But hey, it's good to see you, too, giving great feedback and helping so many others.

  Keep it up! 😀

  1