react-carousel icon indicating copy to clipboard operation
react-carousel copied to clipboard

ScrollingCarousel does not respond to click sometimes

Open jeremylin8 opened this issue 3 years ago • 5 comments
trafficstars

Describe the bug

Hello,

I spot this bug when using ScrollingCarousel component. Sometimes, when click on an item, it doesn't respond to the click event. I think it's caused by the mouseMove and mouseDown event both fired. Could you please take a look at this and fix if that's possible? Thank you so much.

Reproduction

https://user-images.githubusercontent.com/7519225/164739212-6e702305-85f0-45f0-af18-41945e0d5e3a.mov

System Info

System:
    OS: macOS 11.5.1
    CPU: (8) arm64 Apple M1
    Memory: 100.41 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.10.0 - /opt/homebrew/bin/node
    Yarn: 1.22.17 - /opt/homebrew/bin/yarn
    npm: 7.24.0 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 100.0.4896.127
    Firefox: 98.0
    Safari: 14.1.2
  npmPackages:
    @trendyol-js/react-carousel: ^2.0.1 => 2.0.1

Used Package Manager

npm

Logs

No response

Validations

jeremylin8 avatar Apr 22 '22 14:04 jeremylin8

@jeremylin8 did you try the triggerClickOn prop? if you set triggerClickOn={10} it will trigger click events even if you move your mouse 10px. (you should update the version v3.0.0-beta.4)

hasangenc0 avatar Apr 22 '22 17:04 hasangenc0

@hasangenc0 I think triggerClickOn prop is only in Carousel component. Can you guys please add it to the ScrollingCarousel component as well?

jeremylin8 avatar Apr 25 '22 14:04 jeremylin8

I think we can. let's see if we have free time to do it this week.

hasangenc0 avatar Apr 25 '22 17:04 hasangenc0

Hey @hasangenc0, any updates on this feature? Thanks a lot!

jeremylin8 avatar Jun 08 '22 14:06 jeremylin8

+1. Running into the same issue with ScrollingCarousel

GautamGupta avatar Jun 17 '22 01:06 GautamGupta