MAESTRO: Standardize wizard screen padding, spacing, and typography

- Unified keyboard hints styling across all wizard screens using consistent
  mt-4 margin, flex gap-6 layout, and theme.colors.border for kbd elements
- ConversationScreen: Fixed kbd background from bgMain to border to match
  other screens, updated margin from mt-3 to mt-4
- PhaseReviewScreen: Replaced simple text hints with styled kbd-based hints
  matching AgentSelectionScreen and DirectorySelectionScreen patterns
- PhaseReviewScreen: Standardized header/footer padding from px-4 py-3 to
  px-6 py-4 for visual consistency with ConversationScreen sections
- PhaseReviewScreen: Updated content area and error margins to use px-6
This commit is contained in:
Pedram Amini
2025-12-10 03:59:52 -06:00
parent 98c0ecc93d
commit 71d3167054
2 changed files with 46 additions and 14 deletions

View File

@@ -629,14 +629,14 @@ export function ConversationScreen({ theme }: ConversationScreenProps): JSX.Elem
</div>
{/* Keyboard hints */}
<div className="mt-3 flex justify-center gap-6">
<div className="mt-4 flex justify-center gap-6">
<span
className="text-xs flex items-center gap-1"
style={{ color: theme.colors.textDim }}
>
<kbd
className="px-1.5 py-0.5 rounded text-xs"
style={{ backgroundColor: theme.colors.bgMain }}
style={{ backgroundColor: theme.colors.border }}
>
Enter
</kbd>
@@ -648,7 +648,7 @@ export function ConversationScreen({ theme }: ConversationScreenProps): JSX.Elem
>
<kbd
className="px-1.5 py-0.5 rounded text-xs"
style={{ backgroundColor: theme.colors.bgMain }}
style={{ backgroundColor: theme.colors.border }}
>
Shift+Enter
</kbd>
@@ -660,7 +660,7 @@ export function ConversationScreen({ theme }: ConversationScreenProps): JSX.Elem
>
<kbd
className="px-1.5 py-0.5 rounded text-xs"
style={{ backgroundColor: theme.colors.bgMain }}
style={{ backgroundColor: theme.colors.border }}
>
Esc
</kbd>

View File

@@ -1050,7 +1050,7 @@ function DocumentReview({
>
{/* Header with document info */}
<div
className="px-4 py-3 border-b"
className="px-6 py-4 border-b"
style={{
borderColor: theme.colors.border,
backgroundColor: `${theme.colors.success}10`,
@@ -1090,7 +1090,7 @@ function DocumentReview({
</div>
{/* Document editor */}
<div className="flex-1 overflow-hidden p-4">
<div className="flex-1 overflow-hidden px-6 py-4">
<DocumentEditor
content={localContent}
onContentChange={handleContentChange}
@@ -1109,7 +1109,7 @@ function DocumentReview({
{/* Error message */}
{launchError && (
<div
className="mx-4 mb-2 px-4 py-2 rounded-lg flex items-center gap-2"
className="mx-6 mb-2 px-4 py-2 rounded-lg flex items-center gap-2"
style={{
backgroundColor: `${theme.colors.error}20`,
borderColor: theme.colors.error,
@@ -1144,7 +1144,7 @@ function DocumentReview({
{/* Action buttons */}
<div
className="px-4 py-4 border-t"
className="px-6 py-4 border-t"
style={{
borderColor: theme.colors.border,
backgroundColor: theme.colors.bgSidebar,
@@ -1197,12 +1197,44 @@ function DocumentReview({
</div>
{/* Keyboard hints */}
<p
className="text-xs text-center mt-3"
style={{ color: theme.colors.textDim }}
>
Tab to switch between buttons • Enter to select • Escape to go back
</p>
<div className="mt-4 flex justify-center gap-6">
<span
className="text-xs flex items-center gap-1"
style={{ color: theme.colors.textDim }}
>
<kbd
className="px-1.5 py-0.5 rounded text-xs"
style={{ backgroundColor: theme.colors.border }}
>
Tab
</kbd>
Switch buttons
</span>
<span
className="text-xs flex items-center gap-1"
style={{ color: theme.colors.textDim }}
>
<kbd
className="px-1.5 py-0.5 rounded text-xs"
style={{ backgroundColor: theme.colors.border }}
>
Enter
</kbd>
Select
</span>
<span
className="text-xs flex items-center gap-1"
style={{ color: theme.colors.textDim }}
>
<kbd
className="px-1.5 py-0.5 rounded text-xs"
style={{ backgroundColor: theme.colors.border }}
>
Esc
</kbd>
Go back
</span>
</div>
</div>
</div>
);