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

Remove text color from error output #591

Closed
lightyen opened this issue Dec 13, 2021 · 10 comments
Closed

Remove text color from error output #591

lightyen opened this issue Dec 13, 2021 · 10 comments

Comments

@lightyen
Copy link
Contributor

Is there an option that I can remove ANSI escape code?

image

@ben-rogerson
Copy link
Owner

Hey there,
There's no option on this, but looks like a bug I'd need to fix.
What's your tailwind.config.js that generates this bad output?

@lightyen
Copy link
Contributor Author

@ben-rogerson my tailwind config is nothing:

module.exports = {}

@RohitM-IN
Copy link

tailwind v3 update is yet to come u need to wait till then
the cyan color scheme is in v3
check #588

Duplicate of #589

@ben-rogerson
Copy link
Owner

@RootAndroid58 I believe this issue is more about the poor output rather than the missing cyan color here.

@ben-rogerson
Copy link
Owner

@lightyen Twin uses chalk for the terminal colors - I'll update the package in the next version, hopefully that may fix this output.

@lightyen
Copy link
Contributor Author

lightyen commented May 11, 2022

I found that ansi-html doesn't support 24-bit color format, something like: \x1b[38;2;0;0;0mhelloworld\x1b[0m.

Then I have made a package to fix that:
image

@ben-rogerson
Copy link
Owner

Sounds interesting!
Would you be providing a PR I could take a look at?

@lightyen
Copy link
Contributor Author

@ben-rogerson
Copy link
Owner

So in that example you fixed the display of the error overlay, that's always been an issue - nice one 👍

I'm wondering how we can bring this fix into some of the twin examples.
How could we use ansi-html-string to fix the overlay on react-emotion-typescript?

@ben-rogerson
Copy link
Owner

Closing due to inactivity - feel free to reopen if there's something to act on here

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

3 participants