JunoLab / atom-ink

IDE toolkit for Atom
MIT License
229 stars 40 forks source link

ANSI colors missing from console #175

Closed mauricioszabo closed 6 years ago

mauricioszabo commented 6 years ago

The new version of Ink broke ANSI escape codes from Ink console, like the screenshot below:

image

As we're working with Clojure, running code inside editor, and sometimes diffs are shown on console, this is a big deal for me right now. I've checked that there's now two "console" folders on lib, and "console" still has the code for ANSI codes, but not on console2.

I'm using INK 0.9.6, on Atom 1.30.0, on Ubuntu 16.04

pfitzseb commented 6 years ago

Fixed with https://github.com/JunoLab/atom-ink/commit/18a708394698c6a23b9204d401d627599b969d16.

Could proto-repl switch to the new console implementation (in the console2 dir), which is based on xterm.js? This one probably won't be maintained for much longer.

pfitzseb commented 6 years ago

New version is out, so this should work again. Please reopen if not.

royvandewater commented 6 years ago

I'm still seeing the issue after updating to 0.9.7

pfitzseb commented 6 years ago

Actually fixed in https://github.com/JunoLab/atom-ink/commit/0d9621d55bcb044af9d8b242a8c747410a770806.

New release should be out soonish.

royvandewater commented 6 years ago

I'm sorry, it looks like version 0.9.8 is still showing the escaped ANSI color codes instead of colorizing the output.

pfitzseb commented 6 years ago

Sigh... I think I've finally found a good test case (because the old code does work in most cases, at least for how Juno uses it), so this should be fixed for good now. Can you check the latest commit (should be as easy as apm dev ink and then start atom -d)? If it works fine for you I'll tag another release immediately.

royvandewater commented 6 years ago

I'll try that right away

royvandewater commented 6 years ago

It worked! Thank you for your patience with this, that had to be pretty frustrating!

pfitzseb commented 6 years ago

Ok, new version should be out :) Thanks for testing!

And yeah, me being so obtuse was kinda frustrating... :P