From 830d3613519c7c1d4e2e1ca8264ed50a8cd7f7bd Mon Sep 17 00:00:00 2001 From: librelad Date: Wed, 17 Jun 2026 17:57:26 +0100 Subject: [PATCH] fix(overview): drop redundant Check button from empty Improvements state MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Improvements tab's empty state ('No hotfix data yet …') rendered an inline 'Check now' button. It was redundant: the embedding Overview header already carries a manual Check, and the host-side auto-scan repopulates the signed improvements index on its own within a couple of minutes (the empty message already says so). Remove the button so the empty state is just the self-explanatory, automation-backed message. Co-Authored-By: Claude Opus 4.8 Signed-off-by: librelad --- .../frontend/components/updater/js/updater-page.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/containers/libreportal/frontend/components/updater/js/updater-page.js b/containers/libreportal/frontend/components/updater/js/updater-page.js index f625788..7c0c72e 100644 --- a/containers/libreportal/frontend/components/updater/js/updater-page.js +++ b/containers/libreportal/frontend/components/updater/js/updater-page.js @@ -342,7 +342,10 @@ class UpdaterPage { // withToolbar=false lets an embedding surface (the fleet Overview tab) skip // the inline Check button because it provides one in its own header. renderImprovements(withToolbar = true) { - if (!this.artifacts) return this.empty('No hotfix data yet — the automatic scan fetches the signed improvements index within a couple of minutes.', true); + // No inline Check button on the empty state: the host auto-scan repopulates + // this within a couple of minutes (and the embedding header already carries + // a manual Check), so the message alone is the right, button-free empty UI. + if (!this.artifacts) return this.empty('No hotfix data yet — the automatic scan fetches the signed improvements index within a couple of minutes.'); const list = Array.isArray(this.artifacts.artifacts) ? this.artifacts.artifacts : []; const signed = !!this.artifacts.signed; if (!list.length) return this.empty('No improvements available right now — you are all caught up. 🎉');