Added code element with code highlighter.

[butschster]

I decided not to use code from nunomaduro/collision package and copied Code Highlighter with some modifications.

1. I got rid of ConsoleColor class and used css classes for syntax highlighting
2. Replaced $linesBefore, $linesAfter with $startLine

<code line="20" start-line="14">
    &lt;?php

    /** @test */
    function sentryReport()
    {
        try {
            throw new \Exception('Something went wrong');
        } catch (\Throwable $e) {
            report($e);
        }
    }
</code>

!!! Users have to convert all applicable characters to HTML entities inside code element !!!

Will render

fixes #82

[butschster]

@xiCO2k Let's start to discuss this PR :)))

[xiCO2k]

Hey @butschster will check later today. sorry for the delay.

[xiCO2k]

Just tried and found this issue:

check the line 23 on the printscreen.

the code i used:

render(<<<'HTML'
    <code line="20" start-line="14">
        <?php

        /** @test */
        function sentryReport()
        {
            try {
                throw new \Exception('Something went wrong');
            } catch (\Throwable $e) {
                report($e);
            }
        }
    </code>
HTML);

[xiCO2k]

Ok I changed the encoding and it works well, there is any way to not need to convert the chars?

[butschster]

Ok I changed the encoding and it works well, there is any way to not need to convert the chars?

Unfortunately no ((( DOMNode always tries to make HTML valid during export, that's why html inside code element should be converted to html entities.

[xiCO2k]

Yes you are right. will check the changes, and let you know right after. Thanks.

[xiCO2k]

Also why not rename the toElement method and call it render like we have on the HtmlRenderer?

Thanks, sorry for the delay reviewing it.

[butschster]

@xiCO2k I like toElement more because this method returns Element object. But if you like render more it's not a problem!

[xiCO2k]

Lets keep the toElement.

[nunomaduro]

@butschster Find some issues.

1. If no "line" is specified it chooses the line 1. Maybe no line at all?
2. I feel that the left spacing between the code an the code numbers should be removed.

[butschster]

@butschster Find some issues.

1. If no "line" is specified it chooses the line 1. Maybe no line at all?
2. I feel that the left spacing between the code an the code numbers should be removed.

Yeah, I thought about it. Is there any chance to find out how many spaces we should get rid of?!

Look at this code example:

<code>
 
             public function test() {
                   .....
             }

             public function test1() {
                   .....
             }

      }
?>
</code>