Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(Slider): avoid triggering the slide event in Swpier #5937

Closed
wants to merge 0 commits into from

Conversation

miracles1919
Copy link
Contributor

#5894

在不同的设备上,useDrag 返回的 event 不同,Pointer/Touch/Mouse

尝试通过 triggerAllEvents: true,给所有相关的 event 加上 stopPropagation

@codecov
Copy link

codecov bot commented Jan 12, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (da348c7) 90.38% compared to head (3aa6b69) 90.39%.

❗ Current head 3aa6b69 differs from pull request most recent head 5fbcaf0. Consider uploading reports for the commit 5fbcaf0 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5937   +/-   ##
=======================================
  Coverage   90.38%   90.39%           
=======================================
  Files         300      300           
  Lines        6513     6517    +4     
  Branches     1616     1617    +1     
=======================================
+ Hits         5887     5891    +4     
  Misses        575      575           
  Partials       51       51           
Impacted Files Coverage Δ
src/components/slider/thumb.tsx 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions
Copy link
Contributor

PR preview has been successfully built and deployed to https://antd-mobile-preview-pr-5937.surge.sh

@zombieJ
Copy link
Member

zombieJ commented Jan 12, 2023

突然想到,如果就是需要冒泡的话这样就直接被吞了。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants