From df6c8e34ac117fa9f3657134bf6b1821d774bd41 Mon Sep 17 00:00:00 2001 From: Arnas Donauskas Date: Fri, 22 May 2026 16:25:22 +0300 Subject: [PATCH] =?UTF-8?q?fix(flows):=20create-flow=20dialog=20was=20stil?= =?UTF-8?q?l=20clamped=20to=20384px=20=E2=80=94=20override=20sm:=20variant?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR #129 added `max-w-3xl` to the create-flow DialogContent thinking it would widen the modal. It was a no-op. shadcn's DialogContent has `sm:max-w-sm` baked into its default classes, and Tailwind-merge treats `max-w-*` (base) and `sm:max-w-*` (sm-prefixed) as different variants — both survive, and at every real desktop breakpoint the sm-scoped 384px wins. So the modal stayed at 384px regardless. Template cards continued to look squished because the dialog never actually grew. Override at the sm: variant directly. Use 4xl (896px) — gives each of the three template cards roughly 270px of breathing room, enough for the description to wrap to 4-5 lines instead of 10+. Left a comment at the call site so the next person doesn't repeat the same mistake. Co-Authored-By: Claude Opus 4.7 (1M context) --- src/app/(dashboard)/flows/page.tsx | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/app/(dashboard)/flows/page.tsx b/src/app/(dashboard)/flows/page.tsx index aff327a..e037bd5 100644 --- a/src/app/(dashboard)/flows/page.tsx +++ b/src/app/(dashboard)/flows/page.tsx @@ -245,7 +245,11 @@ export default function FlowsPage() { )} - + {/* `sm:max-w-4xl` not `max-w-4xl` — shadcn's DialogContent has + `sm:max-w-sm` baked into its default classes. Without the + sm: prefix our override applies at base only and the + sm-scoped 384px wins at every real desktop breakpoint. */} + Create a new flow