-
Notifications
You must be signed in to change notification settings - Fork 166
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
fix player panel #645
Conversation
我们实际上并不进行 canary 发布,似乎没有必要维护一个单独的 dev 分支 我想知道在 F-Droid 管线中,我们可以设置 tag 来拉取代码,而不是始终拉取 main 分支吗,这可能才是正确的做法是 我们可以暂时不合并这个 PR 直到 gitlab 那边的 MR 完成,但我不知道这需要多久 |
可以拉取 tag,checkupdate 应该是拉取的 tag 对应的 commit ID 不过貌似是始终拉取 default 分支 |
我们预期的双击效果是什么样的。修改后的代码行为似乎也和 B 站不同。 B 站在控制面板为显示时,双击无论触发播放还是暂停都不会显示控制面板。 此外你觉得我们现在还需要等待 F-Droid 相关 MR 的合并吗 |
@bggRGjQaUbCoE 修改了移动端双击暂停的逻辑。 尽管这是我的推测,看起来你是 macOS 用户(因为你可以测试 iOS 代码)。你如果愿意可以在 #515 中发表一下你的看法 |
|
不然我们还是改成和 B 站一致吧。B 站的 UI/UX 是专业设计的。 |
43507ce
to
5480867
Compare
现在双击不会影响面板了,锁定屏幕时不注册双击回调,与 B 站保持一致 一并改了手势快进快退。现在横向滑动屏幕会呼出面板,并根据滑动前的面板状态,在结束滑动时选择直接隐藏面板或者重置 hideTimer,也是 B 站的做法,只不过 B 站在滑动时只会显示进度条隐藏其他内容,这就感觉没什么必要了,代码也会变的非常乱 |
为了防止 F-Droid 又出什么意外或许可以在主仓库增加一个开发 branch,然后 PR merge 到开发 branch里,在更新版本时进行 cherry-pick / merge 到 main。虽说 gitlab CI 已经通过了,应该不会有什么意外了。
因为并没有发布 pre-release 和 cherry-pick 新功能之类的问题,专门的开发 branch 倒也是可有可无。