flexbox css and flickity for carousel

Community feedback

• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffe

  Posted over 4 years ago

  Hey, Negi-Rohit! 👋

  Good work on this challenge! Using Flickity for the carousel is cool! 😎

  I suggest,

  • Setting overflow-x: hidden to the body (or something similar) to prevent a horizontal scroll bar from appearing along the bottom of the screen when the page is resized.
  • Making sure some of the background images and the logo in the footer of the page don't distort when the width of the screen changes and/or becomes smaller.
  • The box-shadow around the mobile navigation is quite strong. Making it more subtle would be a nice design touch, I think.
  • Don't forget to make your form accessible to clear up your solution report!

  Keep coding (and happy coding, too)! 😁

  21
• 

  Jade Buniel•185

  @jadebuniel

  Posted over 4 years ago

  Great use of flickity library for the carousel. I was also doing this challenge and got stuck on the carousel part. One thing I noticed tho is horizontal scrollbars are present when scrolling through the carousel

  11