Skip to content

Comments

fix: only refresh related gallery row#15900

Merged
tobiasKaminsky merged 1 commit intomasterfrom
fix/refreshing-gallery-adapter-fully
Nov 5, 2025
Merged

fix: only refresh related gallery row#15900
tobiasKaminsky merged 1 commit intomasterfrom
fix/refreshing-gallery-adapter-fully

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Nov 4, 2025

  • Tests written, or not not needed

Changes

No need to refresh whole adapter for marking file as favorite or during multi-select

Before

Screen.Recording.2025-11-04.at.12.07.40.mov

After

Screen.Recording.2025-11-04.at.12.04.36.mov

Signed-off-by: alperozturk <alper_ozturk@proton.me>
@alperozturk96
Copy link
Collaborator Author

/backport to stable-3.34

@github-actions
Copy link

github-actions bot commented Nov 4, 2025

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/15900.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions
Copy link

github-actions bot commented Nov 4, 2025

Codacy

Lint

TypemasterPR
Warnings4142
Errors1111

SpotBugs

Lint increased!

SpotBugs increased!

@alperozturk96 alperozturk96 added the performance 🚀 Performance improvement opportunities (non-crash related) label Nov 4, 2025
@tobiasKaminsky tobiasKaminsky merged commit 978c574 into master Nov 5, 2025
20 of 23 checks passed
@tobiasKaminsky tobiasKaminsky deleted the fix/refreshing-gallery-adapter-fully branch November 5, 2025 09:23
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.

3 participants