Javascript Slider | Testimonials | Mobile Friendly

Community feedback

• 

  Shashi Lo•1,345

  @shashiloPosted over 4 years ago

  Good job Alan. Visually, this is close to the design. I like the hover state for the arrows. It's very subtle but effective.

  You are missing some details within the design and I'd restructure your display styles.

  1. Image is not in the correct position based off the background image.
  2. Image is missing border-radius and the box-shadow is too heavy.
  3. Font weight and color do not match the design.
  4. If you wrap a .container around you section with a. max-width, it will be easier to control the position of your columns. Also, I'd avoid using position: absolute in this instance. This can be accomplished using just Flexbox.
  5. On mobile, the second slide's title is missing.

  1
• 

  Alan•335

  @alanhcrdzPosted over 4 years ago

  Thank you! it's great to have your feedback, looks like I've yet missed some details.

  0