From a0de12eeb1ad0249f4b7f099eb0f3fd24c8e4bf8 Mon Sep 17 00:00:00 2001 From: jelveh Date: Sat, 18 Jul 2026 14:31:24 -0700 Subject: [PATCH] fix: show a message instead of a blank pane when a Dashboard folder fails to open renderDirectory empties the file list before awaiting readdir, so a readdir rejection (permission lost, folder deleted from another session, network error) left a completely blank pane with only a console error. Render a centered "This folder couldn't be opened." message on the catch path. Verified live: a simulated readdir failure shows the message, clears the spinner, and leaves navigation working (next folder loads). --- src/gui/src/UI/Dashboard/TabFiles.js | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/gui/src/UI/Dashboard/TabFiles.js b/src/gui/src/UI/Dashboard/TabFiles.js index 7aa1a2671..848a31ebb 100644 --- a/src/gui/src/UI/Dashboard/TabFiles.js +++ b/src/gui/src/UI/Dashboard/TabFiles.js @@ -1855,6 +1855,21 @@ const TabFiles = { // network). Without this, renderingDirectory would stay true and // the guard above would block all further navigation. console.error('Failed to read directory:', err); + // The container was already emptied above; show a message instead of + // leaving a blank pane with no explanation. + this.$el_window.find('.files-tab .files').html(`
This folder couldn't be opened.
`); this.hideSpinner(); this.renderingDirectory = false; return;