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

Can we allow text colour to be changed as well? #86

Closed
ChadTaljaardt opened this issue Jan 30, 2020 · 5 comments
Closed

Can we allow text colour to be changed as well? #86

ChadTaljaardt opened this issue Jan 30, 2020 · 5 comments

Comments

@ChadTaljaardt
Copy link

I don't see in the documentation any way to change the colour of the text for the spinner. Is there a way this can be done?

@briandowns
Copy link
Owner

Hi @ChadTaljaardt, it doesn't appear to be a way once the spinner has started. I can take a look at adding feature unless you wanted to submit a PR. :)

@ChadTaljaardt
Copy link
Author

Do you know of a way before the spinner has started? I am looking to have either the prefix text or suffix text to be coloured.

I can look into making a PR on the weekend if i am free.

@theckman
Copy link

theckman commented May 4, 2020

@ChadTaljaardt because of this exact issue I ended up rolling my own spinner starting at the end of the year, yacspin.

I had looked at contributing this functionality to this package, but with the way the spinner was structured it felt like I'd need to make some major changes to do it in a thread-safe way. That made me yak shave my own CLi spinner, which probably ended up being more work. 😅

@brycesteinhoff
Copy link

You can set Spinner.Suffix to pre-stylized string.

@briandowns
Copy link
Owner

Closing for now. Prestyled strings in suffix would be preferred at this time.

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

No branches or pull requests

4 participants