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:修复横滑组件在快速滚动时的闪动 #225

Closed
wants to merge 1 commit into from

Conversation

davedavehong
Copy link

@davedavehong davedavehong commented Dec 6, 2018

问题现象

iOS真机下,多个带有操作入口的横滑组件(AtSwipeAction)在快速上下滑动时,会闪现即将进入页面的横滑组件的操作入口。

问题定位

iOS真机下,对view组件设置transform: translate3d(0,0,0)的样式,会导致view组件出现闪现情况。

问题修复

在没有横向拖动的交互时,不设置transform样式,为了保证上下滚动时尽量不要频繁出发横滑,将偏移量的敏感值设置为20

@achefinil
Copy link

这个官方日后会发布吗

@davedavehong
Copy link
Author

这个官方日后会发布吗

都还没合并呢

@SzHeJason
Copy link
Collaborator

横滑组件~~ 我还以为是 Tab组件呢 稍等我看下

@SzHeJason
Copy link
Collaborator

#192 实际上跟这个是一个问题 与 transform: translate3d 属性无关 是偏移量的问题 目前你的方案有点粗 我会争取在下个版本处理一个新方案的 如果这个阶段有一样的问题的同学 可以暂时参考这个

@davedavehong
Copy link
Author

#192 实际上跟这个是一个问题 与 transform: translate3d 属性无关 是偏移量的问题 目前你的方案有点粗 我会争取在下个版本处理一个新方案的 如果这个阶段有一样的问题的同学 可以暂时参考这个

我打印日志看过,新出现的横滑组件的偏移量都是0,但是还是会出现这个问题。我在业务场景下试过,把你们的代码放在公司私有库再这么修改是可以的,这个问题也只会在iOS真机上出现,你们再品品吧

@SzHeJason SzHeJason closed this in a671632 Dec 18, 2018
jimczj added a commit that referenced this pull request Dec 19, 2018
* 'dev' of github.com:nervjs/taro-ui:
  fix(FloatLayout): 暂时屏蔽 onClose 回应问题 close #266
  fix(SwipeAction): 处理 SwipeAction 的一些问题 close #225
@JoMartinezZhu
Copy link

修改 了 Taro-ui 源码 ,体验 👌了,但是怎么 放在 公司的私有库?

@SzHeJason
Copy link
Collaborator

@JoMartinezZhu ? 新版的 taro-ui 应该已经修复这个问题了

@JoMartinezZhu
Copy link

JoMartinezZhu commented Dec 25, 2018

@JoMartinezZhu ? 新版的 taro-ui 应该已经修复这个问题了

真机测试,是修复了,但是只有前几个 可以 左滑动,但是其他 不能滑动!
在 scroll-view 中 ,上下滑动时,安卓出现抖动(轻微卡顿)!
我已经更新到 最新版本了,taro-ui(1.5.3)

@SzHeJason
Copy link
Collaborator

好的 我看看

@SzHeJason
Copy link
Collaborator

@JoMartinezZhu 微信小程序 并没有重现你说的问题

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.

4 participants