Skip to content

fix(files): update loading behavior when changing location - #63166

Open
luka-nextcloud wants to merge 3 commits into
masterfrom
fix/files-list-loading
Open

fix(files): update loading behavior when changing location#63166
luka-nextcloud wants to merge 3 commits into
masterfrom
fix/files-list-loading

Conversation

@luka-nextcloud

@luka-nextcloud luka-nextcloud commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Improves the loading state handling and user experience when navigating in the files list. Ensure that loading indicators are displayed appropriately and that the UI does not show stale or confusing content during transitions.

Checklist

@luka-nextcloud
luka-nextcloud requested review from a team and skjnldsv as code owners August 11, 2026 17:03
@luka-nextcloud
luka-nextcloud requested review from kristian-zendato and susnux and removed request for a team August 11, 2026 17:03
@luka-nextcloud
luka-nextcloud force-pushed the fix/files-list-loading branch from 8a27ba8 to 01cba5d Compare August 11, 2026 17:04
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Aug 11, 2026
@luka-nextcloud luka-nextcloud moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team Aug 11, 2026

@susnux susnux 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.

This is a sensible area of changes, please add proper tests for the changes behavior (playwright). Moreover this was designed behavior back then to not block users from usage during long running loading. For example the search view should not blank just because the search is in progress.

So besides that point this must get UX approval from designers, because IMHO this is now worse UX than the behavior it tries to fix.

Currently:

  • Folder contents A
  • navigate
  • Folder contents A + loading indicator
  • Folder contents B

With this changes:

  • Folder contents A
  • navigate
  • Blank view + loading indicator
  • Folder contents B

@luka-nextcloud
luka-nextcloud force-pushed the fix/files-list-loading branch from 01cba5d to dfa612f Compare August 24, 2026 07:43
@luka-nextcloud
luka-nextcloud requested a review from susnux August 24, 2026 07:45
@susnux

susnux commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

But this still blanks the table content and shows the loading spinner within the table.
This is not what was said on the issue, the loading spinner within the table is only for new views or initial load.
But when changing folders the top bar loading spinner should be used to be still able to cancel the loading by interacting with the list:

grafik

Ideally with the "de-saturation" of the table content as shown in the mockup by @kra-mo

@joshtrichards joshtrichards added design Design, UI, UX, etc. 2. developing Work in progress feature: files labels Aug 28, 2026
Signed-off-by: Luka Trovic <luka@nextcloud.com>
…tion

Signed-off-by: Luka Trovic <luka@nextcloud.com>
Signed-off-by: Luka Trovic <luka@nextcloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress design Design, UI, UX, etc. feature: files

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

UX Improvement: Files app: The files grid should be blur or disabled when navigating

3 participants