Slider (Vanilla CS & JS)

Community feedback

• 

  Marlon Passos•940

  @MarlonPassos-gitPosted about 3 years ago

  Very cool your solution, practically a pixel perfect.

  particularly the only thing I could suggest to you is to add a stronger style to the clickable button, so it's easier for the user to know that there are more images, maybe adding a cursor: not-allowed to the disabled button would be nice. Another thing I see on sites with carousel is the possibility of changing the image using the keyboard arrows, maybe you can implement that too 😄

  1

  ---

  

  Eray•1,410

  @ErayBarslanPosted about 3 years ago

  @MarlonPassos-git Thank you for the review :)

  I thought of making another button icon that's darker but went with what's provided by starter files in the end. Will definitely do that for larger projects. And adding arrow keys was in my plan forgot to implement, thanks for reminding.

  1