Skip to content

Video player: fullscreen now hides system status bar, keep screen on while video is playing - #17830

Open
daniele-verducci wants to merge 2 commits into
masterfrom
fix/17825-video-player-timeout-fullscreen
Open

daniele-verducci wants to merge 2 commits into
masterfrom
fix/17825-video-player-timeout-fullscreen

Conversation

@daniele-verducci

@daniele-verducci daniele-verducci commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Fix: #17825

This change:

  • keeps screen on while playing video
  • hides the top status bar when in fullscreen (the system bars were hidden, but the status bar remained visible all the time)

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

…d of leaving the top system bar in place

Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
@codecov

codecov Bot commented Oct 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...wncloud/android/ui/preview/PreviewImageActivity.kt 0.00% 2 Missing ⚠️
...loud/android/ui/preview/PreviewPlaybackFragment.kt 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@github-actions

Copy link
Copy Markdown

📱 QA build

Download app-qa-debug.apk
QR code Open the QR code for this download
Commit bf6a0ce
Version 17830
Available until 7 days after this build

The QA build installs alongside a released Nextcloud app, so you can keep
using your existing install while testing.

Downloading the file requires a GitHub account, so open this link on the
device you want to test on, or transfer the APK to it.

@github-actions

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice2525
Correctness5252
Dodgy code161161
Experimental11
Malicious code vulnerability11
Multithreaded correctness2020
Performance2828
Security1616
Total304304

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Video player does not go fullscreen and screen turns off during playback (Galaxy Z Fold6, 35.0.1)

1 participant