Skip to content

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

Merged
alperozturk96 merged 2 commits into
masterfrom
fix/17825-video-player-timeout-fullscreen
Oct 19, 2026
Merged

alperozturk96 merged 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

@alperozturk96
alperozturk96 merged commit 0ae8a0b into master Oct 19, 2026
22 of 25 checks passed
@alperozturk96
alperozturk96 deleted the fix/17825-video-player-timeout-fullscreen branch October 19, 2026 07:31
@alperozturk96

alperozturk96 commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

This cleans-up naming: #17840

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)

2 participants