Reword the warning and link it to the FAQ entry

This commit is contained in:
Dave Pearson
2023-03-07 11:51:08 +00:00
parent 56ba7d9cc7
commit ff532227d6

View File

@@ -49,6 +49,7 @@ def _post_run_warnings() -> None:
import platform import platform
from rich.console import Console from rich.console import Console
from rich.panel import Panel
console = Console() console = Console()
@@ -56,14 +57,14 @@ def _post_run_warnings() -> None:
( (
platform.system() == "Darwin" platform.system() == "Darwin"
and os.environ.get("TERM_PROGRAM") == "Apple_Terminal", and os.environ.get("TERM_PROGRAM") == "Apple_Terminal",
"The default terminal app is limited to 256 colors. We recommend installing a newer terminal " "The default terminal app on macOS is limited to 256 colors. See our FAQ for more details:\n\n"
"such as iTerm2, Kitty, or WezTerm.", "https://github.com/Textualize/textual/blob/main/FAQ.md#why-doesn't-textual-look-good-on-macos",
) )
] ]
for concering, concern in warnings: for concering, concern in warnings:
if concering: if concering:
console.print(f"[bold yellow]{concern}[/]") console.print(Panel.fit(f"⚠️ [bold green] {concern}[/]", style="cyan"))
@run.command( @run.command(