Skip to content

Add primary ripple for full screen exoplayer controls#17054

Open
AndyScherzinger wants to merge 4 commits into
masterfrom
style/noid/exoPlayerControllsRipple
Open

Add primary ripple for full screen exoplayer controls#17054
AndyScherzinger wants to merge 4 commits into
masterfrom
style/noid/exoPlayerControllsRipple

Conversation

@AndyScherzinger
Copy link
Copy Markdown
Member

AI-assistant: Claude Code v2.1.141 (Claude Sonnet 4.6)

🖼️ Screenshots

🏚️ Before 🏡 After
B A

🏁 Checklist

  • Tests written, or not not needed

AI-assistant: Claude Code v2.1.141 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Claude Code v2.1.141 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@AndyScherzinger AndyScherzinger force-pushed the style/noid/exoPlayerControllsRipple branch from 6026fa4 to 6ecebac Compare May 15, 2026 11:09
@AndyScherzinger AndyScherzinger changed the title style: Add primary ripple for full screen exoplayer controls Add primary ripple for full screen exoplayer controls May 15, 2026
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
…ple' into style/noid/exoPlayerControllsRipple

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@github-actions
Copy link
Copy Markdown

blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed.

@github-actions
Copy link
Copy Markdown

APK file: https://github.com/nextcloud/android/actions/runs/25914910892/artifacts/7018728885
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@github-actions
Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice3535
Correctness6767
Dodgy code216216
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3434
Performance4242
Security1616
Total420420

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants