hatch config show terminal colors don't seem compatible with light terminals about hatch HOT 9 CLOSED

Can you please tell me what happens when you change the background color here? https://github.com/pypa/hatch/blob/hatch-v1.10.0/src/hatch/cli/config/__init__.py#L36

from hatch.

Using "white"

        text = app.config_file.read() if all_keys else app.config_file.read_scrubbed()
        app.output(Syntax(text.rstrip(), 'toml', background_color='white'))

produces

Using '#ffffff'

        text = app.config_file.read() if all_keys else app.config_file.read_scrubbed()
        app.output(Syntax(text.rstrip(), 'toml', background_color='#ffffff'))

produces

Using None

        app.output(Syntax(text.rstrip(), 'toml', background_color=None))

produces

That's readable. Not pretty, but the contrast works, so it's readable. 🙂

from hatch.

@willmcgugan What should I do here? I'm not sure what is recommended.

from hatch.

Syntax will use an Pygments theme by default. Pygments themes use RGB colors , which means that changing the background without changing the text colors is unlikely to look good.

You can also use ansi colors, which will render with the user's choice of terminal theme colors. Just set theme="ansi_light" or theme="ansi_dark".

from hatch.

I would suggest to keep it simple, because programs like gcc, rustc, ls, rg etc output colorful text without trying to know which background color the terminal has.

Pick a syntax coloring with few classes (few different colors or styles) and use ansi colors, I guess?

from hatch.

I'm just going to remove the explicit background color for broader compatibility even though on light terminals it might not be as pretty like you mentioned. If you require configurability (and can provide example keyword arguments that look good) feel free to open a separate feature request!

from hatch.

Just as a heads up (mostly for Will) the background_color option must be set to default for tests because otherwise a bunch of spaces get written at the end of each line I think based on the terminal size. That may be in fact why I did that originally 😅

from hatch.

Ok, thanks for that. What do you think about a config option that turns off syntax highlight for this?

from hatch.

Sure I can do that.

from hatch.