Skip to content

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Oct 20, 2025

  • Tests written, or not not needed

Changes

Handles empty list state correctly in FolderPickerActivity

How to reproduce?

  1. Open the Media tab.
  2. Click the More button in the top-right corner and select Set Media Folder.
  3. Navigate between folders. (Occasionally, the interface may become unresponsive and remain in a loading state.)

@alperozturk96 alperozturk96 marked this pull request as draft October 20, 2025 13:17
@alperozturk96 alperozturk96 force-pushed the add/idle-empty-list-state branch from 3e31c59 to ce23a08 Compare October 21, 2025 08:42
@alperozturk96 alperozturk96 changed the title add: idle empty list state fix: folder picker activity empty list state Oct 21, 2025
@alperozturk96 alperozturk96 force-pushed the add/idle-empty-list-state branch from ce23a08 to d2fc248 Compare October 21, 2025 08:57
@alperozturk96 alperozturk96 force-pushed the add/idle-empty-list-state branch from d2fc248 to a9a00f0 Compare October 21, 2025 08:58
// in owncloud library with broadcast notifications pending to process
DataHolderUtil.getInstance().delete(intent.getStringExtra(FileSyncAdapter.EXTRA_RESULT))
} finally {
listOfFilesFragment?.setEmptyListMessage(EmptyListState.LOCAL_FILE_LIST_EMPTY_FILE)
Copy link
Collaborator Author

@alperozturk96 alperozturk96 Oct 21, 2025

Choose a reason for hiding this comment

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

If ExtendedListFragment contains items, empty list state is not visible thus adding IDLE state in this PR out of scope. Maybe we can add via separate PR.

user,
applicationContext
)
operation.execute(
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Previously, setBackgroundText() is called before waiting completion of the RefreshFolderOperation.

@nextcloud nextcloud deleted a comment from github-actions bot Oct 21, 2025
@alperozturk96 alperozturk96 marked this pull request as ready for review October 21, 2025 09:28
@github-actions
Copy link

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

@alperozturk96
Copy link
Collaborator Author

/backport to stable-3.34

@alperozturk96 alperozturk96 merged commit a5c7b87 into master Oct 31, 2025
18 of 23 checks passed
@alperozturk96 alperozturk96 deleted the add/idle-empty-list-state branch October 31, 2025 14:20
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