From ca381957a7de0c5d9be1f44bb8b5ae0ecc7bcc16 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= Date: Sat, 16 May 2026 01:47:42 +0200 Subject: [PATCH] fix(coach): align mobile sheet height to the responsive-sheet convention MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Coach drawer's phone branch capped at 95 dvh, leaving only a 5 % slice of the underlying /insights page visible — read as a takeover instead of a sheet. Every other mobile sheet in the app routes through , whose phone branch caps at 90 dvh so 10 % of the underlying page stays visible. Match the convention here so the Coach drawer feels like the same kind of surface as the rest of the app on phone-class viewports. --- .../insights/coach-panel/coach-drawer.tsx | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/src/components/insights/coach-panel/coach-drawer.tsx b/src/components/insights/coach-panel/coach-drawer.tsx index ed696cb26..7f3d68983 100644 --- a/src/components/insights/coach-panel/coach-drawer.tsx +++ b/src/components/insights/coach-panel/coach-drawer.tsx @@ -288,11 +288,14 @@ export function CoachDrawer({ "w-full p-0 sm:max-w-[720px]", "lg:!max-w-[min(960px,75vw)] xl:!max-w-[1080px]", isPhoneViewport - ? // Bottom-sheet caps at 95 dvh so a sliver of the underlying - // /insights page remains visible — clear "this is a - // sheet, not a takeover" signal. The rounded top corners - // match the iOS bottom-sheet feel. - "flex h-[95dvh] max-h-[95dvh] flex-col gap-0 rounded-t-2xl" + ? // Bottom-sheet caps at 90 dvh so a 10 % slice of the + // underlying /insights page stays visible — same + // convention as every other mobile sheet on the app via + // 's phone branch (v1.4.28 R3c BK-M5 + // alignment; previously 95 dvh, which left only a 5 % + // sliver and read as a takeover instead of a sheet). + // Rounded top corners match the iOS bottom-sheet feel. + "flex h-[90dvh] max-h-[90dvh] flex-col gap-0 rounded-t-2xl" : "flex h-[100dvh] flex-col gap-0", )} >