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

修复重力旋转全屏 #1068

Merged
merged 6 commits into from
Oct 20, 2024
Merged

修复重力旋转全屏 #1068

merged 6 commits into from
Oct 20, 2024

Conversation

NieR4ever
Copy link
Member

fix #686

@Him188 Him188 self-requested a review October 14, 2024 17:10
@NeKoOuO
Copy link
Contributor

NeKoOuO commented Oct 15, 2024

要不要順便改一下描述詞 重力感應不夠白話 改成自動旋轉螢幕會比較好

@Him188
Copy link
Member

Him188 commented Oct 15, 2024

要不要順便改一下描述詞 重力感應不夠白話 改成自動旋轉螢幕會比較好

separate pr

@Him188 Him188 force-pushed the NieR/fix-gravity-fullscreen branch from b6d4f87 to 16625f4 Compare October 17, 2024 11:12
@Him188 Him188 added p: android 平台: Android s: player 子系统: 视频播放器 labels Oct 17, 2024
@Him188 Him188 added this to the 3.13.0 milestone Oct 17, 2024
@NieR4ever NieR4ever marked this pull request as draft October 18, 2024 00:47
@Him188
Copy link
Member

Him188 commented Oct 19, 2024

怎么 draft 了

@NieR4ever
Copy link
Member Author

现在的 pr 是有问题的,关闭重力感应全屏会使原来的全屏逻辑无法隐藏状态栏,需要处理好 Activity 因为屏幕方向改变而重启的问题

@NieR4ever
Copy link
Member Author

NieR4ever commented Oct 19, 2024

其实感觉全屏逻辑(重力和全屏按钮)并不需要自己隐藏状态栏,应该是在 onResume 里根据 isFullscreen 的状态来判断当前状态栏要不要隐藏

这样的话就不是重力感应全屏的问题了

@NieR4ever NieR4ever marked this pull request as ready for review October 19, 2024 05:15
Copy link
Member

@Him188 Him188 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK

  • Mi14, HyperOS Android 14

@Him188 Him188 enabled auto-merge (squash) October 19, 2024 19:14
@Him188 Him188 merged commit a018a06 into main Oct 20, 2024
6 checks passed
@Him188 Him188 deleted the NieR/fix-gravity-fullscreen branch October 20, 2024 09:36
StageGuard pushed a commit that referenced this pull request Oct 22, 2024
* fix ScreenRotationEffect

* Revert "fix ScreenRotationEffect"

This reverts commit 16625f4.

* use setSystemBarVisible

* Update app/shared/src/commonMain/kotlin/ui/subject/episode/EpisodePage.kt

* remove suspend

* remove debug mode

---------

Co-authored-by: Him188 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p: android 平台: Android s: player 子系统: 视频播放器
Projects
None yet
Development

Successfully merging this pull request may close these issues.

#646 的自動全屏不會自動收縮上方的通知欄
3 participants