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

docs(readme): fix underscore colours guide #646

Merged
merged 1 commit into from
Oct 22, 2024
Merged

Conversation

lljbash
Copy link
Contributor

@lljbash lljbash commented Oct 21, 2024

Description

According to this answer, \e[58:2:R:G:Bm is considered a "rarely used incorrect format" for truecolor.

I tested it on Windows Terminal with WSL2, and it didn’t work. After changing the format to \e[58::2:R:G:Bm, which is noted as the "commonly used standard way," it worked as expected.

The change I made is as follows:

- set -as terminal-overrides ',*:Setulc=\E[58::2::%p1%{65536}%/%d::%p1%{256}%/%{255}%&%d::%p1%{255}%&%d%;m'
+ set -as terminal-overrides ',*:Setulc=\E[58::2::::%p1%{65536}%/%d::%p1%{256}%/%{255}%&%d::%p1%{255}%&%d%;m'

I’d appreciate it if someone could test this on more platforms.

Related Issue(s)

Screenshots

@lljbash lljbash changed the title doc: fix underscore colours guide docs: fix underscore colours guide Oct 21, 2024
@lljbash lljbash changed the title docs: fix underscore colours guide docs(readme): fix underscore colours guide Oct 21, 2024
@folke folke merged commit c75935b into folke:main Oct 22, 2024
3 of 5 checks passed
@folke
Copy link
Owner

folke commented Oct 22, 2024

ty

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.

2 participants