In convert() method, if detect \r, erase the entire line :

replace: $text = preg_replace('#^.*\r(?!\n)#m', '', $text);

by: $text = preg_replace('#^(.*)\r(?!\n)#m', '$1', $text);

| Q | A | | --- | --- | | Bug Fix? | no | | New Feature? | no | | BC Breaks? | no | | Deprecations? | no | | Tests Pass? | yes | | Fixed Tickets | | | License | MIT |

I had a problem with displaying results is Sismo.

( ! ) Notice: Undefined variable: text in Sismo/vendor/sensiolabs/ansi-to-html/SensioLabs/AnsiConverter/AnsiToHtmlConverter.php on line 112

It turns out that converting underline to html isn't properly done so i made this fix.

I thing in the README.md the following line :

$twig->addExtension(AnsiExtension($converter));

should be corriged to :

$twig->addExtension(new AnsiExtension($converter));

AnsiExtension is a class and not a function

My command prints a bunch of lines using writeln function, but the ansi-to-html converter does not make separate lines out of them. Is there a way to achieve this?

Hello,

What do you think about using rollup to produce minified version of the library to have a browser-ready build? I do this following suggestions from https://www.pika.dev/search?q=ansi-to-html

Thanks,

Hi, I did a git clone of the project. How do I integrate it into my project? with the composer autoloader? I get 'Uncaught error: Class 'AnsiToHtmlConverter' not found when I run it.

There's special escape sequences (for VT100) that alter character sets. These don't get replaced, like the cursor movements do.

Escape sequences such as:

\e(B

http://ascii-table.com/ansi-escape-sequences-vt-100.php

You can replicate the issue with the use of the command tput sgr0.

For one of my project, I had to create a twig extension. Do you want a PR?

<?php

use SensioLabs\AnsiConverter\AnsiToHtmlConverter;

class AnsiExtension extends \Twig_Extension
{
    private $converter;

    public function __construct(AnsiToHtmlConverter $converter = null)
    {
        $this->converter = $converter ?: new AnsiToHtmlConverter();
    }

    public function getFilters()
    {
        return array(
            new \Twig_SimpleFilter('ansi_to_html', array($this, 'ansiToHtml'), array('is_safe' => array('html'))),
        );
    }

    public function ansiToHtml($string)
    {
        return $this->converter->convert($string);
    }

    public function getName()
    {
        return 'ansi';
    }
}

It would be nice to add a twig function ansi_css that will ouput the inline CSS. But to do that, we need AnsiToHtmlConverter::getTheme

Some terminals seem to not issue the complete end sequence '[0m' for returning to default color, but using a shorthand '[m', which can be fixed with a little tweak of the regular expression.

<?php

/*
 * This file is part of ansi-to-html.
 *
 * (c) 2013 Fabien Potencier
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace SensioLabs\AnsiConverter\Theme;

/**
 * Hacker theme.
 *
 * @see http://ethanschoonover.com/solarized
 */
class HackerTheme extends Theme
{
    public function asArray()
    {
        return array(
            // normal
            'black' => '#000000',
            'red' => '#800000',
            'green' => '#008000',
            'yellow' => '#808000',
            'blue' => '#000080',
            'magenta' => '#800080',
            'cyan' => '#008080',
            'white' => '#808080',

            // bright
            'brblack' => '#606060',
            'brred' => '#FF0000',
            'brgreen' => '#00FF00',
            'bryellow' => '#FFFF00',
            'brblue' => '#0000FF',
            'brmagenta' => '#FF00FF',
            'brcyan' => '#00FFFF',
            'brwhite' => '#FFFFFF',
        );
    }
}

Here's the fix:

        foreach ($options as $option) {
            if ($option >= 30 && $option < 38) {
                $fg = $option - 30;

            } elseif ($option >= 40 && $option < 48) {
                $bg = $option - 40;
            } elseif ($option >= 90 && $option < 98) { // This needs to be added
                $fg = $option - 80; // This needs to be added
            } elseif (39 == $option) {

... }

We found some devices (H3C S10508-V) may output like this:

blahblahblan \r\r\n
blahblahblan \r\r\n

However the history version may covert it into this:

\r\n
\r\n

I came across a weird behavior while using the library - if background color (e.g. 42 - green) is specified before foreground (e.g. 30 - black) the render will contain black text on black background. Weirdly enough this happens only for separated sequences (e.g. ^[[0m^[[42m^[[30m) and not while parsing their short forms (e.g. ^[[42;30m).

I used following example to demonstrate the problem (also attached): Download raw

^[[30;42mWORKS1^[[0m
^[[0m^[[42m^[[30mDOESNT1^[[0m
^[[0m^[[30m^[[42mWORKS2^[[0m
^[[42;30mWORKS1^[[0m

Rendering in GNU bash, version 4.3.30(1)-release (x86_64-pc-linux-gnu)

HTML from ansi-to-html v1.1.3

<?php
$output = (new AnsiToHtmlConverter())->convert($test);

<span style="background-color: green; color: black">WORKS1</span><span style="background-color: black; color: white"><br />
</span><span style="background-color: black; color: black">DOESNT1</span><span style="background-color: black; color: white"><br />
</span><span style="background-color: green; color: white">WORKS2</span><span style="background-color: black; color: white"><br />
</span><span style="background-color: green; color: black">WORKS1</span><span style="background-color: black; color: white"><br />
</span>

The CSS prefix that is part of the theme takes precedence over the prefix supplied to the converter.

Deprecates $prefix parameter from Theme::asCss()

It would be cumbersome to support a dynamic prefix at this late stage of the rendering.

Tests updated to cover the following combinations.

1. Using the default theme or a supplied theme, with and without its own css prefix.
2. Using the default css prefix or a supplied prefix.
3. Using inline styling or CSS.

Hopefully fixes https://github.com/sensiolabs/ansi-to-html/issues/12