From ce6b2c1816d520d7d1a382cb079b16426b458a53 Mon Sep 17 00:00:00 2001
From: Joey Yakimowich-Payne <jyapayne@pm.me>
Date: Sun, 19 Apr 2026 16:52:08 -0600
Subject: [PATCH] style(ui): floor editor modal height at 85vh so empty state
 isn't flat

Both ModifierProfileEditor and LayoutEditor already capped at max-h-[95vh]
but had no min-height, so they collapsed to just their content when empty
(no modifiers yet, no profiles saved). That looks like a flat toolbar
strip floating over the board rather than a proper editor dialog.

Add min-h-[85vh] to both so the dialog commits to a reasonable stage
regardless of content, and the user immediately understands it's a
full editor modal.
---
 packages/chess/src/ui/LayoutEditor.tsx          | 2 +-
 packages/chess/src/ui/ModifierProfileEditor.tsx | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/chess/src/ui/LayoutEditor.tsx b/packages/chess/src/ui/LayoutEditor.tsx
index 666175a..dd2fb3a 100644
--- a/packages/chess/src/ui/LayoutEditor.tsx
+++ b/packages/chess/src/ui/LayoutEditor.tsx
@@ -269,7 +269,7 @@ export function LayoutEditor({
         if (e.key === 'Escape') onClose();
       }}
     >
-      <div className="bg-white rounded-2xl shadow-2xl w-full max-w-6xl max-h-[95vh] overflow-hidden flex flex-col">
+      <div className="bg-white rounded-2xl shadow-2xl w-full max-w-6xl min-h-[85vh] max-h-[95vh] overflow-hidden flex flex-col">
         {/* Header */}
         <header className="flex items-center justify-between px-6 py-4 border-b border-neutral-200">
           <div className="flex items-center gap-3">
diff --git a/packages/chess/src/ui/ModifierProfileEditor.tsx b/packages/chess/src/ui/ModifierProfileEditor.tsx
index 6356d44..825c773 100644
--- a/packages/chess/src/ui/ModifierProfileEditor.tsx
+++ b/packages/chess/src/ui/ModifierProfileEditor.tsx
@@ -197,7 +197,7 @@ export function ModifierProfileEditor({ isOpen, onClose }: Props) {
       data-testid="modifier-editor-modal"
       className="fixed inset-0 z-50 bg-black/60 backdrop-blur-sm flex items-center justify-center p-4"
     >
-      <div className="bg-white rounded-2xl shadow-2xl w-full max-w-6xl max-h-[95vh] overflow-hidden flex flex-col">
+      <div className="bg-white rounded-2xl shadow-2xl w-full max-w-6xl min-h-[85vh] max-h-[95vh] overflow-hidden flex flex-col">
         {/* Header */}
         <header className="flex items-center justify-between px-6 py-4 border-b border-neutral-200">
           <div className="flex items-center gap-3 min-w-0 flex-wrap">