Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support all xterm variants with color #692

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

relistan
Copy link

@relistan relistan commented Jul 22, 2024

I see from the history that how to support color or not has been changed a few times. Originally it supported any TERM var that contained xterm. Then it was looking at TERM for 256. Now it looks for color.

Some modern terminals, e.g. Kitty, have their own terminal type in order to support extended integration between the shell and the terminal. Kitty's default terminal type is xterm-kitty. I could hack around this by renaming it, however, I feel like goconvey should probably support modern terminals like this.

Probably the better strategy at this point is to have an env var that disables color. However, I did not attempt to mess with that here.

I've been using xterm since at least 1992 and AFAIK it has supported color since well before then. goconvey is not using colors beyond the standard range (e.g. not using 256 color range). So I'm proposing here that goconvey also support color on any terminals with a TERM var containing xterm. This should not affect the behavior of anyone with other terminals on which color is currently supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant