[BUG] "cocoindex setup" title colors don't show well under bright background

[badmonster0]

Under Colab:

Under terminal with bright background:

[badmonster0]

It seems hard to detect background color in tty. To solve this, one way is to pick colors working for both light and dark themes.

For now, probably leaving default colors to titles will just work.

[par4m]

I guess there are a few workarounds for this,

1. "guessing" if $TERM or $COLORTERM looks like it belongs to a light theme
2. Ask the user to set an environment variable (e.g., TERMINAL_THEME=dark) and adapt based on that.
3. Provide a CLI flag like --dark-theme or --light-theme. (if they want, otherwise default to current setup)

I personally think a cli flag is the most optimal solution for this, we can add multiple themes this way since owo-colors supports setting XTerm colors

[par4m]

For now a theme that's neutral to both light and dark themes can be used. Any suggestions ?

[badmonster0]

I feel we should keep the control minimum. It's tedious for users to provide another flag or environment variable, especially new users don't know how to set it. The default appearance should be good enough for most users.

The current color themes:

1. The status like [ READY ] - this parts work well for both light and dark.
2. Headings (the line to the right of the status) like CocoIndex Metadata Table and Flow: DocToKG. This should be more obvious than details (TODOs below), but under light background it's hard to be seen.
3. Details (the TODOs) - this also seems good. Under both light and dark, it's less eminent than others.

So overall I think maybe it's good enough to leave default colors to "2" - then it'll be more eminent than "3" under both dark and light.

[par4m]

Also I realized that the colors that are enforced just take the shade that the terminal set for that color,
e.g

Is this the default behavior that you expect or would you want to enforce a specific e.g Cyan shade ? (setting XTerm color)

[badmonster0]

This looks pretty good. Thanks for trying out!

[par4m]

Those are the defaults colors only, it's just that AnsiColors that are set using owo-colors, my terminal theme set Cyan to that specific shade hence its showing it like that.

Would you want to enforce the Cyan and other color shades or let them be the way the user has them in their terminal theme

[par4m]

Some screenshots with different terminal emulators with different themes

So right now it entirely depends on the theme the user has, meaning it would still use Cyan and other colors we set but the terminal sets the shade!

[badmonster0]

I thought leaving default color to headings is the easiest way, and the appearance looks good to me.

Do you also get a chance to test on light background? (e.g. I'm using Mac's default terminal, and the "Basic" profile has white background)
As long as we can make it look good under both light and dark themes, it's great.

BTW, there's a subtle color difference between the line of Flow: TextEmbedding and other headings (like Tracking Table). The color for Flow: TextEmbedding should be the same with other headings.

[par4m]

I'll test on multiple terminal with multiple themes and experiment a bit, will post screenshots here so whichever looks good we can use that.

[par4m]

So for description we can completely get rid of the color on it and let the terminal set it as it picks up the foreground color of it

This will ensure that it's visible regardless of theme

[badmonster0]

Looks good to me! Thanks for testing out!