From c48f9f4ffc136c7a21d16771404a2e0f1e300f8a Mon Sep 17 00:00:00 2001 From: librelad Date: Thu, 25 Jun 2026 13:26:51 +0100 Subject: [PATCH] =?UTF-8?q?fix(webui/storage):=20drop=20redundant=20'Recla?= =?UTF-8?q?iming=20space=E2=80=A6'=20toast?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Clicking Reclaim space fired two notifications: routeAction → executeTask already shows the rich 'Reclaim Space task started!' toast (icon + bold LibrePortal + task link), then _reclaim added a second, plain, iconless 'Reclaiming space…' info toast on top of it. The image-removal path doesn't double-notify like this. Drop the redundant one — the start toast plus the completion toast give clean feedback on their own. Co-Authored-By: Claude Opus 4.8 Signed-off-by: librelad --- .../components/admin/system/js/system-storage-page.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/containers/libreportal/frontend/components/admin/system/js/system-storage-page.js b/containers/libreportal/frontend/components/admin/system/js/system-storage-page.js index b891a20..92d2ddb 100644 --- a/containers/libreportal/frontend/components/admin/system/js/system-storage-page.js +++ b/containers/libreportal/frontend/components/admin/system/js/system-storage-page.js @@ -123,8 +123,10 @@ class SystemStoragePage { if (!window.tasksManager?.router?.routeAction) { throw new Error('Task system not ready'); } + // routeAction → executeTask already shows the rich "Reclaim + // Space task started!" toast (and a completion toast lands when + // it finishes), so no extra info notification here. await window.tasksManager.router.routeAction('system_reclaim'); - window.notificationSystem?.show?.('Reclaiming space…', 'info'); // The prune runs in the background task processor; re-read // usage a couple of times so the numbers settle without a // manual refresh.