From fdab4e3d84e5be616f695e9b2612084379197f28 Mon Sep 17 00:00:00 2001 From: silverwind Date: Wed, 28 Jun 2023 15:38:55 +0200 Subject: Add custom ansi colors and CSS variables for them (#25546) Use our existing color palette to map to the 16 basic ansi colors. This is backwards-compatible because it aliases the existing color names. Side note: I think the colors in `console.css` for console file rendering are incomplete, but fixing those is out of scope here imo. Before and after: Screenshot 2023-06-28 at 00 26 12 Screenshot 2023-06-28 at 00 28 24 --------- Co-authored-by: Giteabot --- web_src/js/render/ansi.js | 1 + web_src/js/render/ansi.test.js | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) (limited to 'web_src/js/render') diff --git a/web_src/js/render/ansi.js b/web_src/js/render/ansi.js index 68e8f2c544..75b8cb4fd2 100644 --- a/web_src/js/render/ansi.js +++ b/web_src/js/render/ansi.js @@ -11,6 +11,7 @@ export function renderAnsi(line) { // the output of future renders, because ansi_up is stateful and remembers things like // unclosed opening tags for colors. const ansi_up = new (AnsiUp.default || AnsiUp)(); + ansi_up.use_classes = true; if (line.endsWith('\r\n')) { line = line.substring(0, line.length - 2); diff --git a/web_src/js/render/ansi.test.js b/web_src/js/render/ansi.test.js index 7542ba0ede..b4793251df 100644 --- a/web_src/js/render/ansi.test.js +++ b/web_src/js/render/ansi.test.js @@ -8,7 +8,7 @@ test('renderAnsi', () => { expect(renderAnsi('\r')).toEqual(''); expect(renderAnsi('\rx\rabc')).toEqual('x\nabc'); expect(renderAnsi('\rabc\rx\r')).toEqual('abc\nx'); - expect(renderAnsi('\x1b[30mblack\x1b[37mwhite')).toEqual('blackwhite'); // unclosed + expect(renderAnsi('\x1b[30mblack\x1b[37mwhite')).toEqual('blackwhite'); // unclosed expect(renderAnsi('