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

feat: add ANSI colours #98

Merged
merged 19 commits into from
Oct 26, 2024
Merged

feat: add ANSI colours #98

merged 19 commits into from
Oct 26, 2024

Conversation

sgoudham
Copy link
Contributor

@sgoudham sgoudham commented Oct 20, 2024

This PR overhauls our ANSI colours by separating the "normals" and the "brights", note that this doesn't solve the readability issue raised in catppuccin/catppuccin#1961 as that will require more thought and analysis.

By landing this PR first, we can get the infrastructure in for easy updates to all of our terminal ports as the chain of updates are as follows: palette.json -> catppuccin/rust -> catppuccin/whiskers -> updates in all terminal whiskers templates.

For the colours themselves, we're adjusting the lightness, chroma and hue. Credit to @unseen-ninja for originally mocking up the jsfiddle/CSS that this is based on. However, I believe we need to adjust the formula as the library used (color.js) here is outputting different hex codes than the original mockup.

Leaving this as draft until we're happy with the final results.

@sgoudham sgoudham marked this pull request as ready for review October 23, 2024 00:06
mod.ts Outdated Show resolved Hide resolved
mod.ts Outdated Show resolved Hide resolved
Copy link
Member

@unseen-ninja unseen-ninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After a lot of back and forth over the past couple of months I'm pretty happy with the results here. c:

While this doesn't resolve all of the issues that initially sparked this while ANSI thing I still think that the right way is to adhere to the standard and use the colours people would expect instead of diverting from it to potentially fix issues that stem from the fact that Catppuccin's colours don't necessarily work with every ANSI-standard interface out there. Especially with how text should be coloured on accent-background.

@sgoudham
Copy link
Contributor Author

Thanks for the approval. I'm happy to merge this after I update the README 👍

Copy link
Member

@backwardspy backwardspy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome

@sgoudham sgoudham merged commit 1fdc227 into main Oct 26, 2024
1 check passed
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.

3 participants