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

[feature]能否为SketchZoomImageView增加手势旋转功能 #169

Closed
RavenLiao opened this issue Sep 18, 2022 · 2 comments
Closed

[feature]能否为SketchZoomImageView增加手势旋转功能 #169

RavenLiao opened this issue Sep 18, 2022 · 2 comments

Comments

@RavenLiao
Copy link

SketchZoomImageView功能很完善,但唯独缺少了手势旋转功能,只能使用命令进行90°旋转。能否参照PhotoViewEx增加手势旋转(放手后归90°倍数角度)的功能?

@panpf
Copy link
Owner

panpf commented Sep 19, 2022

接下来会考虑这个功能的

@panpf
Copy link
Owner

panpf commented Sep 20, 2022

先说结果:放弃增加手势旋转功能。

原因有以下几点:

  1. 工作量大。尝试实现的过程中发现手势旋转功能实现起来非常复杂,因为不同的角度缩放倍数和位移量都不一样,旋转的过程中要动态计算并更新,现有架构需要较大的重构才能实现。而 PhotoViewEx 并没有考虑这些,所以它的效果并不好
  2. 体验了 PhotoViewEx 的手势旋转后个人感觉手势旋转手势用起来体验并不好,远不如提供单独旋转按钮用起来方便舒服
  3. 纵观市面上排名靠前的 App,例如微信、微博、QQ,哔哩哔哩、Twitter它们都没有提供手势旋转功能,甚至他们都没有提供旋转功能,说明这是一个非常低频且不是那么重要的功能

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

No branches or pull requests

2 participants