taro icon indicating copy to clipboard operation
taro copied to clipboard

Fix/h5/movable

Open baosiqing opened this issue 3 years ago • 3 comments

这个 PR 做了什么? (简要描述所做更改) 1、transform样式中丢失了y的单位 2、touch事件不灵敏,将事件绑定改为直接绑定在host节点上,不再加入子节点,此种方式会有一个问题,当用户在movable-view上也绑定touch事件时,用户绑定的touch事件和组件内部的touch事件执行顺序会有问题,第一次是用户绑定的事件先触发,后续的事件都是组件内部的事件先触发,引起这个问题的原因是现在使用的react桥接stencil绑定事件机制引起,可后续看下是否有好的方式调整,当用户碰到此问题时,可以在自己绑定的事件中使用set-timeout临时处理 保证每次的执行顺序一致

这个 PR 是什么类型? (至少选择一个)

  • [x] 错误修复(Bugfix) issue: fix #12319

这个 PR 涉及以下平台:

  • [x] Web 平台(H5)

baosiqing avatar Sep 02 '22 11:09 baosiqing

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

taro-bot2[bot] avatar Sep 02 '22 11:09 taro-bot2[bot]

touch 事件不灵敏,将事件绑定改为直接绑定在host节点上,不再加入子节点

这个组件目前还没有测试用例,结构性变更还是先加一下用例比较好一点

此种方式会有一个问题,当用户在movable-view上也绑定touch事件时,用户绑定的touch事件和组件内部的touch事件执行顺序会有问题,第一次是用户绑定的事件先触发,后续的事件都是组件内部的事件先触发

在代码和文档内做一下标注把,记录一下版本和相关问题

引起这个问题的原因是现在使用的react桥接stencil绑定事件机制引起,可后续看下是否有好的方式调整,当用户碰到此问题时,可以在自己绑定的事件中使用set-timeout临时处理 保证每次的执行顺序一致

在 vue 中是否会有类似的问题,改动之后有相关的验证不

ZakaryCode avatar Sep 09 '22 09:09 ZakaryCode

友好的ping.

yoyo837 avatar Sep 15 '22 04:09 yoyo837