SQL code blocks disappear about distill HOT 5 CLOSED

Going to close this and create a new issue covering adding more Prism languages.

from distill.

Source

https://github.com/rstudio/radix/blob/27b482a21b6cba1c21752abdcd6b55ff5e083218/inst/rmarkdown/templates/radix_article/resources/distill.html#L575

Details

The pre element within the d-code block tag is missing for the sql chunk but present for the chunks that render correctly. A child of pre that's supposed to contain the code (code id="code-container") is actually there but

1. it has nothing in it and
2. it doesn't carry a language class

Results for R

<pre>
    <code id="code-container" class="language-clike">
        data <span class="token operator">&lt;</span>
        <span class="token operator">-</span> iris
    </code>
</pre>

Results for Sql

<code id="code-container"></code>

from distill.

Fixed via rstudio/radix@868669f

Our only option here was to map languages not supported directly by Prism to "clike"

from distill.

I'm a little confused, apologies if I'm misunderstanding something at some point. :-)

Prism.js supports 152 languages, including R and SQL: https://prismjs.com/#languages-list

However, looking at the source of template/src/components/d-code.js from distill, they only explicitly import python, clike, lua, bash, go, markdown and julia. So this could be fixed "properly" by importing more languages - but is this something that radix could override, or would it need a pull request to distill?

from distill.

We'd have to remove the embedded Prism from Distill then replace it with another implementation. So far we've taken the approach not to fork/edit Distill but it may be that we'll ultimately need to.

from distill.