Skip to content

Conversation

quentinyang
Copy link
Contributor

As rc-gesture revert preventDefault in v0.0.19 for tabs+listview issue.
Then, rc-gesture@v0.0.20 fixed the swipe issue by exposing the event as a property of Gesture object, and the Gesture object will be passed as the first parameter when invoked panMove event function.
So, in this version, to fix it, rc-swipeout invokes event.preventDefault() to prevent scroll event when pan moving.

@quentinyang quentinyang requested review from warmhug and paranoidjk June 4, 2018 08:20
@quentinyang quentinyang self-assigned this Jun 4, 2018
@quentinyang quentinyang added the bug label Jun 4, 2018
As rc-gesture revert preventDefault in v0.0.19 for tabs+listview issue (ant-design/ant-design-mobile#2589).
Then, rc-gesture@v0.0.20 fixed this issue by exposing the `event` as a property of Gesture object, and the Gesture object will be passed as the first parameter when invoked panMove event callback.
So, `rc-swipeout` invokes `event.preventDefault()` to prevent scroll event when pan moving.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 92.035% when pulling 17b218f on fix-gesture-issue into eba18d3 on master.

@warmhug warmhug requested a review from ziluo June 5, 2018 03:33
@warmhug
Copy link

warmhug commented Jun 5, 2018

我和江木都没多少时间投入这些review测试了

@ziluo 这个 rc-swipeout 和 rmc-tabs 一样都依赖 rc-gesture ,调整下 组件负责人,你和 @quentinyang 一起负责这几个组件 (如果有其他依赖到 rc-gesture 的上层组件 也一起负责掉) ,一起持续的跟踪改动、互相 review 效率高、不容易出问题。

总之,手势相关的都是 非常核心 影响很大 的功能,认真全面地回归测试、OK了你们合并。

Copy link

@ziluo ziluo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@ziluo
Copy link

ziluo commented Jun 6, 2018

这个后续也把测试用例完善下吧

@ziluo ziluo merged commit b6c208e into master Jun 6, 2018
@ziluo ziluo deleted the fix-gesture-issue branch June 6, 2018 02:39
@paranoidjk
Copy link
Member

CI 挂了,把自己加到 Travis 的邮件通知里面去,合 mr之前关注 ci

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

Successfully merging this pull request may close these issues.

哈~ 还是找到bug了, swipeAction滑动锁逻辑有缺陷~~
5 participants