-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Colorized output doesn't follow terminal ANSI color styles. #524
Comments
@winneon There currently is no support for this. HTTPie uses color schemes from Pygments, with the exception of our solarized implementation. There's the
What I believe could be done though is to create a custom theme whose colors would be dynamically assigned from the terminal ANSI color scheme. |
That would be great! I'm not able to make a pull request at the moment because my PC's motherboard is out for replacement, but if there isn't a change implemented when I get it back, then I will take a look into that. |
When I just change the formatter to
|
@dsego any updates on the pr? |
@ivancuric I deleted my fork, there was no interest. |
@dsego do you still have the code around? Completely missed your comment in the ocean of GitHub notifications. |
@jkbrzt Sorry, scrubbed it. I think it was using TerminalFormatter instead of Terminal256Formatter when no style option is chosen. |
@jkbrzt @ivancuric |
So maybe I'm just crazy but it seems like this doesn't work when I choose the |
HTTPie prefers solarized for its color palette, and it seems it doesn't properly follow whatever terminal style / colors that the end-user has set.
For example: I have specific ANSI colors set in my terminals, whether it be Terminal.app or HyperTerm on macOS, and yet, HTTPie still prefers solarized.
Terminal.app:
HyperTerm:
The text was updated successfully, but these errors were encountered: