Only reload file list when switching navigation sections

[PVince81]

Description

Reverts a previous bad fix from #27703 which causes a double reload and even breaks sometimes when opening a private link by reloading the wrong list.

Adds a better fix that only reloads a file list whenever the active navigation section has changed, because that's when we need to refresh, for example after restoring a file from trash, the "All files" list needs to refresh to display the restored folder.

Also prevents initial double reload of the file list.

Related Issue

Fixes #27890

Motivation and Context

How Has This Been Tested?

• create folder "test" in "All files", delete it. go to trash: folder appears
• create folder "test/sub" in "All files". Refresh page several times: folder contents is still correct, no blinking. (before this fix it was either blinking or showing the wrong contents)

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[davitol]

👍

[phil-davis]

Fixes the "flashing load" of the home folder, before then displaying the sub-folder content.

[phil-davis]

Approving because it works. Would be another fix of an annoying little thing that can be got into the next beta maybe.

[PVince81]

stable10: #28843

[guruz]

yay nice. I was annoyed this breaks the scrollTo private linkls.. hope this fixes it

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.