Skip to content

fix(file-list): scroll position#16846

Open
alperozturk96 wants to merge 3 commits intomasterfrom
fix/file-list-scroll-to-top
Open

fix(file-list): scroll position#16846
alperozturk96 wants to merge 3 commits intomasterfrom
fix/file-list-scroll-to-top

Conversation

@alperozturk96
Copy link
Copy Markdown
Collaborator

@alperozturk96 alperozturk96 commented Apr 14, 2026

Issue

Previous scroll position lost

Changes

Removes unused argument
Introduces new Kotlin file for all magic numbers
Fixes scroll position lost
Introduces layout manager aware scrolling

Demo

Screen_recording_20260414_120212.webm

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 added this to the Nextcloud App 33.1.0 milestone Apr 14, 2026
@alperozturk96
Copy link
Copy Markdown
Collaborator Author

/backport to stable-33.1.0

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@nextcloud nextcloud deleted a comment from github-actions bot Apr 14, 2026
@github-actions
Copy link
Copy Markdown

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

Signed-off-by: alperozturk96 <alper_ozturk@proton.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

Codacy

SpotBugs

CategoryBaseNew
Bad practice3535
Correctness6969
Dodgy code227228
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3434
Performance4242
Security1616
Total433434

SpotBugs increased!

org.jetbrains.annotations.* is used. Please use androidx.annotation.* instead.

@github-actions
Copy link
Copy Markdown

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

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.

2 participants