Ping challenge with React and Scss

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, nice job on this challenge, Guy Ezra! 👍

  Your solution looks good and the form validation works well (I like the little notification that pops up after valid input has been submitted)! 👏

  I suggest,

  • Adding cursor: pointer to the social media icons at the bottom of the page and adding a little padding around the icons themselves to separate them from the border around them.
  • Adding some space around the hero image (using margin or padding) so that it isn't right up against the edges of the screen in the mobile layout.

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Guy Ezra•615

  @ezraguyPosted almost 4 years ago

  Hello @ApplePieGiraffe, Always a pleasure getting feedback from you! I've fixed the issues you mentioned. thank you so much! 😄

  1

  

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  @ezraguy

  Just took another look at your solution, and the changes you made look great! Keep it up! 👍

  1