PHPBB3-13919 - Syntax highlighting in [code]

[hanakin]

prismjs is the current standard for highlighting

www.prismjs.com
https://github.com/PrismJS/prism

[Rhino.Freak]

So is it coming in for 3.3? I hope it is?
I tried to implement and currently using Pretty code by Google on my 3.1 test forum.. It sort of works but sort of doesn't. Having it in core would be great.

[hanakin]

not sure if u r referring to me or joshyphp, but as far as prismjs that depends on the new theme. It will probably not be a major thought until its finished, but it will integrate easily enough with some minor changes in the backend

[hanakin]

3.1 provides syntax highlighting for PHP (and only PHP) via highlight_string(), using the BBCode [code=php].

3.2 has no syntax highlighting in code blocks for any language. My recommendation is to use a JavaScript highlighter such as highlight.js or SyntaxHighlighter. My preference goes to highlight.js because it automatically detects the language of the source and it also requires one script tag to be installed. Both are available on various CDNs.

I don't know whether 3.2 should provide syntax highlighting at all. As far as I'm concerned, I wrote an extension for it to see how hard it would be to implement. All it takes is a script block in overall_footer to dynamically load highligher.js whererever needed. It costs 490 bytes per page. Unminified code here.

not sure how the ext works but as it is now with 3.2 it does not produce the code properly from the parsser in that when we use [code=xxx] we do not get the expected result in the html which should be

Code: Select all

<pre>
	<code class="xxx">
	</code>
</pre>

I think this at a minimum should be included into the core so that it can be used with whichever highlighter you wish to use. It should not be a breaking change so could possilby add it to 3.2.x

[JoshyPHP]

The extension only loads highlight.js from a CDN without modifying any BBCodes. highlight.js does its own auto-detection, it doesn't require a language hint although I believe it uses it if present. Is there a reason you want to use the language as class name?

[hanakin]

The extension only loads highlight.js from a CDN without modifying any BBCodes. highlight.js does its own auto-detection, it doesn't require a language hint although I believe it uses it if present. Is there a reason you want to use the language as class name?

I am not familiar with highlight.js , but we should be as agnostic as possible to whichever framework someone wants to use there are many and most require it. Also there are cases that could be made for code blocks that should be highlighted and others they should not

[jesssy]

Simple. With javaScript code prettifier:
http://devcodepro.com/view/34/6/Syntax- ... phpBB-3.2-

[3Di]

Simple. With javaScript code prettifier:
http://devcodepro.com/view/34/6/Syntax- ... phpBB-3.2-

Which is dated 2006, something newer and similar I could look at? I mean, not already mentioned here.

[DavidIQ]

Ummm what?
https://github.com/google/code-prettify

Honestly don't know why this is even being brought up anyways since there is a semi-official extension for syntax highlighting already: https://github.com/s9e/phpbb-ext-highlighter

[3Di]

David, honestly I was looking at https://cdn.rawgit.com/google/code-pret ... rettify.js and I didn't see the 2013 on line 3 at the time I posted, my fault. The github link comes in handy, forked as well.

Well, I know the Joshyphp's extension already, but I am not completely satisfied of the results (not a Joshy's fault indeed).

Honestly don't know why this is even being brought up anyways since there is a semi-official extension for syntax highlighting already:

there are cases that could be made for code blocks that should be highlighted and others they should not

That's why.. https://github.com/3D-I/code-prettify#h ... ed-as-code