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 time display issue on video player #110

Merged
merged 4 commits into from
Aug 4, 2024
Merged

Conversation

quyaoning
Copy link
Contributor

修正前,超过1h的视频时长无法正确显示,如图:
image
修正后显示效果如下:
image

应用退出按钮可见界面右上角。

@Predidit
Copy link
Owner

Predidit commented Aug 4, 2024

感谢您的工作。

  1. 我们是否可以在视频时长小于60分钟时不显示小时,这样更加美观。

  2. 我们是否可以在此页面不添加退出按钮,退出按钮在这里略显突兀,并且我觉得暴露小组件的内部方法到外部是个很糟糕的主意。

@quyaoning quyaoning marked this pull request as draft August 4, 2024 04:15
@quyaoning quyaoning changed the title 修正播放器时长显示bug,并在播放界面添加原本缺失的退出应用按钮 Fix time displayment issue on video player Aug 4, 2024
@quyaoning quyaoning changed the title Fix time displayment issue on video player Fix time display issue on video player Aug 4, 2024
@quyaoning
Copy link
Contributor Author

我已经根据您的提议进行修改
关于在播放页添加退出按钮 我原先只考虑到了windows端。移动端界面出现这样的按钮的确是极其不合理的 但同样,对于windows应用在任何一个界面不具备这样的交互也是不合逻辑的 因为windows用户不能像移动端一样直接通过手势退出应用。
我认为可以检测是否为desktop来显示/隐藏该按钮。
同样 我认为为sys_app_bar添加windows标题栏的 最小化 最大/窗口化 可以提升windows端的交互体验。如果您觉得合适的话我可以对此提交新pr。

@quyaoning quyaoning marked this pull request as ready for review August 4, 2024 05:08
@Predidit
Copy link
Owner

Predidit commented Aug 4, 2024

在这个PR里我们可以只关心播放器进度条显示错误的问题,在我看来这个PR已经没有问题,我会在 CI 完成后进行合并。

关于 SysAppbar 的修改,推荐页右上角的按钮本身已经过多,添加最小化与最大化窗口会使其更加不协调,且最大化窗体会使程序出现布局错误,flutter 的 UI 实际上并不能做到完全的响应式。

@quyaoning
Copy link
Contributor Author

好的 感谢

@Predidit Predidit merged commit 7b080ec into Predidit:main Aug 4, 2024
4 checks passed
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