syntax highlight in traceback view

[mihnatenko]

No description provided.

[mihnatenko]

It works, but not finished yet. Some code should be removed.

To highlight current line I'm splitting source code to lines and wrapping it with span.current. Looks like a hack, but works.
Also, highlight.js doesn't support line numbers, which are useful in this case, I think. So, I can try to add them somehow. Or maybe it would be better to try prinsm.js, which has line numbers and line highlight?

[jettify]

prism.js looks like actively maintained, so why not

[asvetlov]

I've never used prism but was happy with highlight.js.
Anyway, if you guys choose prism.js I have no objections.

[jettify]

@stormandco sorry of delay I will review this PR as soon as resolve issue with interactive debugger

[mihnatenko]

@jettify no problem. I think it needs some cleanup in tbtools.py
Will try to do this on the weekends

[jettify]

tbtools.py very dirty port that handles interactive console logic, tracebacks etc, ported from flask to pyramid_debugtoolbar and then here... I have not touched that much because I managed to make it work. To rewrite that chunk is bit tricky task.

[mihnatenko]

At least class Line, SOURCE_TABLE_HTML, SOURCE_LINE_HTML strings, and some methods in Frame should be removed

[jettify]

sure, if it simplifies code greatly, it would be easer to refactor interactive console as result get rid of thread locals

[mihnatenko]

@jettify @asvetlov
Could you please review this PR?

[asvetlov]

I'll take a look in few days.

[asvetlov]

Sorry for long delay.
Thank you for both the patch and your patience.