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 player panel #645

Merged
merged 3 commits into from
Jan 22, 2025
Merged

fix player panel #645

merged 3 commits into from
Jan 22, 2025

Conversation

ErBWs
Copy link
Contributor

@ErBWs ErBWs commented Jan 22, 2025

为了防止 F-Droid 又出什么意外或许可以在主仓库增加一个开发 branch,然后 PR merge 到开发 branch里,在更新版本时进行 cherry-pick / merge 到 main。虽说 gitlab CI 已经通过了,应该不会有什么意外了。

因为并没有发布 pre-release 和 cherry-pick 新功能之类的问题,专门的开发 branch 倒也是可有可无。

simulator_screenshot_13EA1D21-0EA1-4134-88D2-D7A465AB9C33

@Predidit
Copy link
Owner

我们实际上并不进行 canary 发布,似乎没有必要维护一个单独的 dev 分支

我想知道在 F-Droid 管线中,我们可以设置 tag 来拉取代码,而不是始终拉取 main 分支吗,这可能才是正确的做法是

我们可以暂时不合并这个 PR 直到 gitlab 那边的 MR 完成,但我不知道这需要多久

@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 22, 2025

可以拉取 tag,checkupdate 应该是拉取的 tag 对应的 commit ID

不过貌似是始终拉取 default 分支

@ErBWs ErBWs changed the title fix tablet player panel fix player panel Jan 22, 2025
@Predidit
Copy link
Owner

我们预期的双击效果是什么样的。修改后的代码行为似乎也和 B 站不同。 B 站在控制面板为显示时,双击无论触发播放还是暂停都不会显示控制面板。

此外你觉得我们现在还需要等待 F-Droid 相关 MR 的合并吗

@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 22, 2025

@bggRGjQaUbCoE 修改了移动端双击暂停的逻辑。

尽管这是我的推测,看起来你是 macOS 用户(因为你可以测试 iOS 代码)。你如果愿意可以在 #515 中发表一下你的看法

@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 22, 2025

我们预期的双击效果是什么样的。修改后的代码行为似乎也和 B 站不同。 B 站在控制面板为显示时,双击无论触发播放还是暂停都不会显示控制面板。

此外你觉得我们现在还需要等待 F-Droid 相关 MR 的合并吗

  1. 目前的效果是暂停时双击会隐藏,播放时双击会显示,我看了下 B 站的逻辑是只会影响播放暂停,不会操作面板,我感觉都差不多?

  2. 或许不需要了

@Predidit
Copy link
Owner

不然我们还是改成和 B 站一致吧。B 站的 UI/UX 是专业设计的。

@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 22, 2025

现在双击不会影响面板了,锁定屏幕时不注册双击回调,与 B 站保持一致

一并改了手势快进快退。现在横向滑动屏幕会呼出面板,并根据滑动前的面板状态,在结束滑动时选择直接隐藏面板或者重置 hideTimer,也是 B 站的做法,只不过 B 站在滑动时只会显示进度条隐藏其他内容,这就感觉没什么必要了,代码也会变的非常乱

@Predidit Predidit merged commit e774723 into Predidit:main Jan 22, 2025
6 checks passed
@ErBWs ErBWs deleted the fix-player-panel branch January 22, 2025 16:37
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.

2 participants