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.

Sign up for GitHub

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

Jump to bottom

[bug-report] [Swiper] Swiper freeze while scrolling #540

Closed
LiangNex opened this issue Aug 20, 2019 · 0 comments
Closed

[bug-report] [Swiper] Swiper freeze while scrolling #540

LiangNex opened this issue Aug 20, 2019 · 0 comments
Assignees
@xxyan0205
Labels
bug Something isn't working @Components Components related

Comments

@LiangNex
Copy link

Mand Mobile Version

2.4.1

Vue Version

2.6.1

Model / OS & Browser Info

iphone x

Recurring Links

https://didi.github.io/mand-mobile/#/zh-CN/docs/co

Recurring Steps

Let's say we use the tabBar demo which includes swiper

  • Swipe the swiper item horizontally a few and keep the drag event
  • Swipe vertically, make the vertical distance bigger than horizontal distance

Expectant Behaviors

The swiper roll back to the original index item

Actual Behaviors

The swiper freezes, no rollback

Looks like you are comparing the vertical distance and horizontal distance to judge the relevant event, might be a fix there.
@xxyan0205 xxyan0205 self-assigned this Aug 20, 2019
@xxyan0205 xxyan0205 added @Components Components related bug Something isn't working labels Aug 20, 2019
xxyan0205 added a commit that referenced this issue Aug 20, 2019
@xxyan0205
fix(swiper): refactor user scrolling judgment by touch angle
55310a4 
#540
@xxyan0205 xxyan0205 mentioned this issue Aug 20, 2019
Merged
xxyan0205 added a commit that referenced this issue Sep 4, 2019
@xxyan0205
fix(swiper): refactor user scrolling judgment by touch angle (#541)
af40f5c 
* fix(swiper): refactor user scrolling judgment by touch angle

#540

* test(swiper): add testing code & update test case

* fix(swiper): remove traslating animation in touchmove
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xxyan0205 xxyan0205

Labels
bug Something isn't working @Components Components related
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xxyan0205 @LiangNex