From 8cff425c8d14225040ee704ee73712cf82e3ee31 Mon Sep 17 00:00:00 2001 From: maziggy Date: Thu, 21 May 2026 07:57:22 +0200 Subject: [PATCH] fix(printers): AMS drying popover scrolls instead of clipping the Start button (#1458) #1447 added computePopoverPosition() to flip the drying popover above the trigger when below would overflow. Its degraded branch (popover taller than the viewport) keeps the popover below and its comment promised "the user can scroll inside it" -- but the popover div was overflow-hidden with no max-height, so the Start button stayed unreachable on short viewports. Make the promise true: the popover is now a flex column capped at calc(100vh - top - 8px); the body scrolls (overflow-y-auto) and the header/footer are shrink-0, so the Start button is pinned and always reachable. Covers the MacBook + iPhone 15 Pro Max PWA cases in #1458. --- frontend/src/pages/PrintersPage.tsx | 20 ++++++++++++++----- .../{index-CHKJFwmW.js => index-CY4xdITS.js} | 2 +- static/assets/index-CyXgRo_7.css | 1 + static/assets/index-KYwGxnG9.css | 1 - static/index.html | 4 ++-- 5 files changed, 19 insertions(+), 9 deletions(-) rename static/assets/{index-CHKJFwmW.js => index-CY4xdITS.js} (97%) create mode 100644 static/assets/index-CyXgRo_7.css delete mode 100644 static/assets/index-KYwGxnG9.css diff --git a/frontend/src/pages/PrintersPage.tsx b/frontend/src/pages/PrintersPage.tsx index 1dcc3bdd6..0c6997120 100644 --- a/frontend/src/pages/PrintersPage.tsx +++ b/frontend/src/pages/PrintersPage.tsx @@ -5398,17 +5398,27 @@ function PrinterCard({
setDryingPopoverAmsId(null)} /> {/* Popover */}
e.stopPropagation()} > {/* Header */} -
+
{t('printers.drying.start')}
{/* Body */} -
+
{/* Filament type select */}
@@ -5501,7 +5511,7 @@ function PrinterCard({
{/* Footer */} -
+