New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
enh(parser) simplify nohighlight #2374
Conversation
- Now only `no-highlight` and `nohighlight` skip a block entirely - `plain` will do nothing - `text` is now an alias for `plaintext` Closes highlightjs#2363.
@egor Any ideas on betting naming for I decided to make both configurable while I was in there since it was easy and it's come up in conversation before. |
I hadn't considered uses like (from the removed tests): <pre><code class="text-content javascript">var x = 'foo';</code></pre>
<pre><code class="clipboard-text javascript">var x = 'foo';</code></pre> But think it's still safe to consider them deprecated for the brutal consistency of "nohighlight". And if anyone is seriously affected they can just use the configuration option. |
* (chore) make `noHighlightRe` and `languagePrefixRe` configurable (languageDetectRe now) * (chore) simply 'nohighlight' regex - Now only `no-highlight` and `nohighlight` skip a block entirely - `plain` will do nothing - `text` is now an alias for `plaintext` Closes highlightjs#2363.
No description provided.