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

Fix: enable VT100 sequences on windows #64

Merged
merged 3 commits into from Dec 9, 2019
Merged

Conversation

@glx22
Copy link
Contributor

glx22 commented Nov 22, 2019

[K on my screen were annoying so instead of removing them I enabled support for them. As VT100 sequences are now enabled, color codes can be used too so I re-enabled the warning color and I changed it to bright yellow because it seems normal yellow is the default powershell foreground (even if visually it's some pale white). And I added color to error messages (bright red, because red is hard to read on default blue powershell background)

Commits should probably be squashed, but that can happen when merging.

nml/generic.py Outdated Show resolved Hide resolved
@glx22 glx22 force-pushed the glx22:ansi branch 2 times, most recently from 58cc89f to fa4298d Nov 23, 2019
nml/generic.py Outdated Show resolved Hide resolved
@glx22 glx22 force-pushed the glx22:ansi branch from fa4298d to 5fa0101 Nov 24, 2019
@LordAro
LordAro approved these changes Dec 7, 2019
@planetmaker planetmaker merged commit 31f3bfe into OpenTTD:master Dec 9, 2019
12 checks passed
12 checks passed
Python 3.5 on ubuntu-latest
Details
Python 3.6 on ubuntu-latest
Details
Python 3.7 on ubuntu-latest
Details
Python 3.8 on ubuntu-latest
Details
Python 3.5 on macOS-latest
Details
Python 3.6 on macOS-latest
Details
Python 3.7 on macOS-latest
Details
Python 3.8 on macOS-latest
Details
Python 3.5 on windows-2016
Details
Python 3.6 on windows-2016
Details
Python 3.7 on windows-2016
Details
Python 3.8 on windows-2016
Details
@glx22 glx22 deleted the glx22:ansi branch Dec 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.