Repository navigation
fix(ui): 修复视频页 Hero 动画优化引入的旋转与全屏异常 - #162
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题描述
4111c92(refactor(ui): 优化视频页Hero 动画卡顿)引入后出现两个回归:
原因
该提交为支持 Hero 封面过渡,把 _VideoDetailPageVState 的初始化整体延后到 Future.delayed(启用 Hero 时延迟 300ms),didChangeDependencies 的布局计算也改为延迟执行,期间 build 渲染空壳。这破坏了页面旋转/全屏状态机依赖的两个前提:
修复方案
在保留 Hero 动画优化(重活仍延迟、过渡期渲染空壳)的前提下,将方向/布局状态恢复到首帧同步计算:
影响范围
仅 lib/pages/video/view.dart。