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 #489 (allow vibrate while long press) #601

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

yusuaois
Copy link
Contributor

在长按倍速时为支持的移动设备添加时长为100ms的震动

在我的安卓上面测试可行,但由于我没有果系的,IOS平台无法测试

@yusuaois
Copy link
Contributor Author

未支持 OpenHarmony

@ErBWs
Copy link
Contributor

ErBWs commented Jan 13, 2025

未支持 OpenHarmony

vibration v1.9.0 及以后的版本支持了 ohos,你可以换下版本

@Predidit
Copy link
Owner

我个人不是很喜欢这个主意

B站的长按倍速也是没有震动反馈的

@yusuaois
Copy link
Contributor Author

我个人不是很喜欢这个主意

B站的长按倍速也是没有震动反馈的

也可以 看你的来

@yusuaois
Copy link
Contributor Author

未支持 OpenHarmony

vibration v1.9.0 及以后的版本支持了 ohos,你可以换下版本

关于ohos的话还需要申请权限,但我好像没找到ohos的配置?

@ErBWs
Copy link
Contributor

ErBWs commented Jan 13, 2025

ohos 在我 fork 仓库的分支里

@Predidit
Copy link
Owner

这个 PR 也可以作为草稿,它的改动很小,后续基本不会发生冲突

@yusuaois
Copy link
Contributor Author

这个 PR 也可以作为草稿,它的改动很小,后续基本不会发生冲突

OK

@ErBWs
Copy link
Contributor

ErBWs commented Jan 13, 2025

我个人不是很喜欢这个主意

B站的长按倍速也是没有震动反馈的

会不会是没给震动权限,B站我印象里一直有震动的,最近B站还加了个上滑锁定倍速播放的功能,动不动给我锁倍速(

@yusuaois yusuaois marked this pull request as draft January 13, 2025 11:02
@stevei5mc
Copy link

我个人不是很喜欢这个主意
B站的长按倍速也是没有震动反馈的

会不会是没给震动权限,B站我印象里一直有震动的,最近B站还加了个上滑锁定倍速播放的功能,动不动给我锁倍速(

确实,我看到这个就去体验一下才发现有震动,之前一直没有发现

@Predidit
Copy link
Owner

我一直在使用 Google Play 版 bilibili ,这个版本似乎是没有触感反馈的

@stevei5mc
Copy link

stevei5mc commented Jan 13, 2025

我一直在使用 Google Play 版 bilibili ,这个版本似乎是没有触感反馈的

那我就不清楚了,反正我这的震动触感很轻,不仔细感觉就几乎感觉不出来

@yusuaois yusuaois marked this pull request as ready for review January 13, 2025 14:36
@yusuaois yusuaois marked this pull request as draft January 13, 2025 14:36
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