Desktop-First & Responsive: HTML, SCSS, FLEXBOX, SVG MANIPULATION

Community feedback

• 

  Folarin Akinloye•1,240

  @folathecoderPosted over 3 years ago

  Thank you very much!

  I appreciate your feedback!

  1
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hey, Folarin Akinloye! 👋

  Nice to see you complete another challenge! 😀 Overall, your solution looks good and responds quite well! 👏

  I'd like to suggest,

  • Adding cursor: pointer to the buttons on the page.
  • Turning the social media icons in the footer of the page into links by wrapping each of them in an anchor tag.
  • Allowing the backgrounds of the various sections of the page to fill up the entire width of the screen so that there isn't any white space to the sides of the design on extra-large screens.
  • Taking a look at your solution report and trying to clear up some of the errors that are there in order to make your solution more accessible.

  Of course, keep coding (and happy coding, too)! 😁

  1