From ff532227d65aa872d3f9ad5912898b5c22a7e2f2 Mon Sep 17 00:00:00 2001 From: Dave Pearson Date: Tue, 7 Mar 2023 11:51:08 +0000 Subject: [PATCH] Reword the warning and link it to the FAQ entry --- src/textual/cli/cli.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/textual/cli/cli.py b/src/textual/cli/cli.py index 05a79646d..4ac6da2c3 100644 --- a/src/textual/cli/cli.py +++ b/src/textual/cli/cli.py @@ -49,6 +49,7 @@ def _post_run_warnings() -> None: import platform from rich.console import Console + from rich.panel import Panel console = Console() @@ -56,14 +57,14 @@ def _post_run_warnings() -> None: ( platform.system() == "Darwin" and os.environ.get("TERM_PROGRAM") == "Apple_Terminal", - "The default terminal app is limited to 256 colors. We recommend installing a newer terminal " - "such as iTerm2, Kitty, or WezTerm.", + "The default terminal app on macOS is limited to 256 colors. See our FAQ for more details:\n\n" + "https://github.com/Textualize/textual/blob/main/FAQ.md#why-doesn't-textual-look-good-on-macos", ) ] for concering, concern in warnings: if concering: - console.print(f"[bold yellow]{concern}[/]") + console.print(Panel.fit(f"⚠️ [bold green] {concern}[/]", style="cyan")) @run.command(