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

Add support for gradient colours #154

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

schmorrison
Copy link

Adds a struct which implements TerminalColor interface. Added steps to Render for gradient application to text.

Adds a struct which implements TerminalColor interface.
Added steps to Render for gradient application to text
@muesli muesli added the enhancement New feature or request label Mar 1, 2023
@muesli
Copy link
Contributor

muesli commented Mar 1, 2023

This looks pretty cool, even though it does complicate the common code paths a bit. I guess we could try and separate a bit of the nested rendering code.

@schmorrison
Copy link
Author

Thanks. TBH I totally forgot about this one. I remember that I couldn't figure out a clean entrypoint to accomplish this. It works and passes tests......... but I'm not sure if you have any plans to modify the Render method ever again 👹

@schmorrison
Copy link
Author

What's this? Do I need to do something?

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

Successfully merging this pull request may close these issues.

3 participants