Skip to content

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Nov 17, 2025

  • Tests written, or not not needed

Changes

If the adapter has data or adapter is null, the empty list should not be visible.
There is no need to update the empty view if its state hasn’t changed.
The empty view state must be checked after setting the adapter and via the adapter’s observer.

tobiasKaminsky

This comment was marked as resolved.

Signed-off-by: alperozturk <[email protected]>

# Conflicts:
#	app/src/main/java/com/owncloud/android/ui/fragment/ExtendedListFragment.kt
Signed-off-by: alperozturk <[email protected]>
@alperozturk96 alperozturk96 force-pushed the fix/empty-list-state-visibility branch from f637539 to 8367d15 Compare November 24, 2025 08:05
@github-actions
Copy link

Codacy

SpotBugs

CategoryBaseNew
Bad practice4242
Correctness7373
Dodgy code272271
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3535
Performance4848
Security1818
Total498497

@github-actions
Copy link

@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/15976.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

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

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.

3 participants