-
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
refactor pc player panel #634
Conversation
新的界面真的非常漂亮 目前的代码看上去质量也相当不错 感谢你的工作 |
这个 PR 太庞大了,我可能需要一些时间来审核 |
确实改动很大,我自己看着都头大 |
我们现在还需要在弹幕发送成功后显示 toast 吗 似乎会遮挡面板 |
我们为什么需要在 smallest_panel.dart 中重新复制一遍 sendDanmaku 函数 这个函数不是已经传入 player_item.dart 了吗 |
没删干净( |
抱歉, sendDanmaku 的相关问题是我看错了 |
是会遮挡,感觉可以去掉 |
那我们可以去掉它 Github 网页面板查看这种庞大的 PR 不是很好用,容易滚动到其他的文件 我拉到本地慢慢看 |
我也看错了( diff 太多被折叠了,我直接看到的 video_page 可以考虑利用 GitHub 的 review comment 评论,可以少打点文件名和函数名 |
这个PR在我看来质量很高,已经没有其他的问题 在解决 toast 问题之后我想我们就可以进行合并 |
目前弹幕输入框提示是照搬的 B 站,我总记得以前 B 站弹幕提示是随机的,本来也想写随机但是我想不出写什么好 |
在实际测试编译产物之后,我发现了两个问题
|
|
如果我们实在解决不了 MenuAnchor 相关问题,可以回退之前的实现 |
|
|
动画可以等 flutter 后续更新修复。 MenuAnchor 是现在官方的推荐实现,用来代替 PopUpMenuButton,PopUpMenuButton 目前看来不符合 Material 3 规范。 MenuAnchor 的接口更加完善,可以很方便的支持二级菜单,比如手机端现在就需要用到 MenuAnchor 的二级菜单。视觉效果更加协调。尽管缺失动画但我认为切换到 MenuAnchor 是更好的实现。 目前只有规则管理界面还在使用 PopUpMenuButton。不过那个界面没有什么视觉上的不协调。 实际上上面提到的 issue 中也给出了一种 workaround,用 AnimationController 手动实现展开效果 |
那有的等,虽说 |
要修改的东西非常多,目前只改了 PC 端的界面,先传上来 review 一下,多平台的适配后面再慢慢改,有可能需要复制一份 player_item_panel.dart 单独展示手机端的界面
PopUpMenuButton 官方文档推荐改为使用 MenuAnchor,我试了一下 hover 的 highlight 看着好看了很多,于是我全部改了(
1.mp4
2.mp4