-
Notifications
You must be signed in to change notification settings - Fork 42
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
utf8 export - wrong colors (light gray = white) #173
Comments
I tried #175 by exporting Utf8Ansi of this ANS file: ...but the resulting utf8ans has lost the "bright white", replaced by "light gray": |
For me it looks like below. Tested on xfce4 terminal and termux (Android). |
The top one probably has a fixed (terminal) color palette with white set to gray. I have this option as well on my terminal. Only very few color palette templates are useful for ANSI art. Try to tweak the terminal or try xfce terminal. The bottom one is broken as the current implementation without #175 doesn't define any color at all at some points. That's why it fluctuates between gray and white. If the terminal interpretation of this null color would be white the picture could be correct. But i haven't seen any terminal doing so. Maybe MacOS or some exotic system. |
The strange thing is that, having white set on gray, the bottom case should never show white... or not? |
I tried XFCE Terminal and it looks OK. |
Yes, i think so. It might also help to take a more complex ANSI for testing. I mostly used this to test (just |
Some colors were wrong in xfce terminal. I cannot remember the exact result anymore though. |
Hi,
i think the light gray color is broken (it's actually white) when exporting as utf8. Tested on xfce and gnome terminal ('white on black' colors).
The upper line is 'cat' output from Linux shell.
The lower one is from Moebius.
Looking at the escape codes you're using [0m (reset/remove all attributes) for light gray and [1m (Bold/Bright) for white. These are no actual color definitions indeed. It might work ok though for white but not for 'light gray' (actually it's in sync with 'white' when changing Terminal colors).
The text was updated successfully, but these errors were encountered: