-
Notifications
You must be signed in to change notification settings - Fork 22.6k
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
Remove 'highlight:[]' and 'highlight[]' classes #8368
Conversation
@@ -98,7 +98,7 @@ <h3 id="Firefox_compatibility_notes">Firefox compatibility notes</h3> | |||
|
|||
<p>For example:</p> | |||
|
|||
<pre class="example-bad brush: js highlight[5,8]">let clearKeyOptions = [ | |||
<pre class="example-bad brush: js">let clearKeyOptions = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does there need to be a separator? Not saying anything wrong with this, but it looks a bit odd to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Between brush:
and js
? Technically yes although again I think we accept the no-space version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wbamberg OK. Since the render works and you're happy to clean this up in source if the converted doesn't, I'll merged.
This is part of #8364 .
The
highlight[...]
andhighlight:[...]
classes are not supported in Markdown and the converter will not convert code blocks that contain them.This PR removes them.
One risk with this is that the text refers explicitly to highlighted lines, which then wouldn't make any sense. I've done my best to check that this isn't happening. Also it's worth noting that
highlight
has never worked in Yari.