Skip to content

perf(media): improvement - #17606

Merged
alperozturk96 merged 19 commits into
masterfrom
fix/media-speed
Sep 11, 2026
Merged

alperozturk96 merged 19 commits into
masterfrom
fix/media-speed

Conversation

@alperozturk96

@alperozturk96 alperozturk96 commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • Removes ThumbnailMemoryCache introduced via new media player PR
  • Cancels previous gallery job onViewRecycled
  • Caches button bitmaps
  • Updates adapter only when content actually changed
  • Fixes placeholder size
  • Fixes placeholder icon inner padding
  • Adds loading spinner to the gallery during load
  • Improves scroll perfomance

@alperozturk96 alperozturk96 added this to the Nextcloud App 35.0.1 milestone Sep 10, 2026
@alperozturk96 alperozturk96 added the performance 馃殌 Performance improvement opportunities (non-crash related) label Sep 10, 2026
@alperozturk96

Copy link
Copy Markdown
Collaborator Author

/backport to stable-35.0.x

@alperozturk96
alperozturk96 force-pushed the fix/media-speed branch 4 times, most recently from 3edaf1d to 694b098 Compare September 11, 2026 07:10
@github-actions

Copy link
Copy Markdown

APK file: https://github.com/nextcloud/android/actions/runs/34573192442/artifacts/10189703611
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)

@daniele-verducci daniele-verducci left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey Alper,
very smooth and fast experience!

However, I found a couple of issues:

Placeholder covers video thumbnail:
image

When selecting a video, it lands on the previous one:

  • Open Media
  • Filter videos only
  • Open a video near the end of the list
  • Go back to PIP mode
  • Open a video upper on the list
  • It shows the previous one

(this was probably there since the media enhancement PR, but I didn't catch it before...)

Screen_recording_20260911_104255.mp4

@alperozturk96

Copy link
Copy Markdown
Collaborator Author

Hey Alper, very smooth and fast experience!

However, I found a couple of issues:

Placeholder covers video thumbnail: image

When selecting a video, it lands on the previous one:

  • Open Media
  • Filter videos only
  • Open a video near the end of the list
  • Go back to PIP mode
  • Open a video upper on the list
  • It shows the previous one

(this was probably there since the media enhancement PR, but I didn't catch it before...)

Screen_recording_20260911_104255.mp4

I updated the PR. Seeing video overlay icon on video is expected but ratio of it needs to be fixed and latest commit includes the fix.

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@github-actions

Copy link
Copy Markdown

APK file: https://github.com/nextcloud/android/actions/runs/34587857929/artifacts/10194544775
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
Correctness5858
Dodgy code195194
Experimental11
Malicious code vulnerability22
Multithreaded correctness2626
Performance3737
Security1919
Total373372

@github-actions

Copy link
Copy Markdown

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

@daniele-verducci daniele-verducci left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's all fixed, as far as I can see. And very very fast and smooth scrolling! Thank you! Approved.

@alperozturk96
alperozturk96 merged commit b6928a8 into master Sep 11, 2026
20 of 22 checks passed
@alperozturk96
alperozturk96 deleted the fix/media-speed branch September 11, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review performance 馃殌 Performance improvement opportunities (non-crash related)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants