Skip to content

Commit f56613f

Browse files
Welchost Contributorsclaude
andcommitted
fix(tui): use brand terracotta for modal borders instead of theme blue
Modal boxes used `border: round $panel`, which resolves to a blue-gray (#242f38) in the textual-dark theme - clashing with the terracotta accent buttons inside. Anchor the border to the brand color (#d97757, themes.ACCENT) so it stays consistent and theme-independent. Affects ConfirmModal and GhosttyRequiredModal. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 1e8661d commit f56613f

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

src/welchost/tui/screens/modals.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,8 @@ class ConfirmModal(ModalScreen[bool]):
3939
ConfirmModal { align: center middle; background: $background; }
4040
ConfirmModal #modal-box {
4141
width: 60; max-width: 90%; height: auto; padding: 1 2;
42-
border: round $panel; background: $surface;
42+
/* Brand terracotta (themes.ACCENT), not the theme's blue-gray $panel. */
43+
border: round #d97757; background: $surface;
4344
}
4445
ConfirmModal #modal-msg { padding: 0 0 1 0; }
4546
ConfirmModal #modal-buttons { height: auto; text-align: center; padding: 1 0 0 0; }
@@ -103,7 +104,8 @@ class GhosttyRequiredModal(ModalScreen[None]):
103104
GhosttyRequiredModal { align: center middle; background: $background; }
104105
GhosttyRequiredModal #modal-box {
105106
width: 60; max-width: 90%; height: auto; padding: 1 2;
106-
border: round $panel; background: $surface;
107+
/* Brand terracotta (themes.ACCENT), not the theme's blue-gray $panel. */
108+
border: round #d97757; background: $surface;
107109
}
108110
GhosttyRequiredModal #modal-msg { padding: 0 0 1 0; }
109111
GhosttyRequiredModal #modal-buttons { height: auto; text-align: center; padding: 1 0 0 0; }

0 commit comments

Comments
 (0)