Skip to content

[markdown] Multi-line blockquotes break nested single-line blockquotes

What does this MR do and why?

Feature flag: markdown_corrected_blockquote

Require our multiline block quote to start with a blank and end with a blank line, to make it less ambiguous from something that is simply triple quoted

Good:

>>>
This is a multiline block quote
>>>

Bad:

>>> This
>>> is not valid

Bad:
>>>
This is not valid
>>>

Screenshots or screen recordings

markdown before after
>>> First paragraph
>>>
>>> Second paragraph
>>>
>>> Third paragraph

Screen_Shot_2022-08-25_at_10.20.40_AM

Screen_Shot_2022-08-25_at_10.20.49_AM

> foo
>>> bar
>>>
>>> baz

> boo
>>> far
>>>
>>> faz

Screen_Shot_2022-08-25_at_10.26.43_AM

Screen_Shot_2022-08-25_at_10.26.51_AM

Something
>>>
quoted
>>>

Screen_Shot_2022-08-25_at_10.30.06_AM

Screen_Shot_2022-08-25_at_10.30.16_AM

A blank line is now required before the blockquote

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #15624 (closed)

Edited by Brett Walker

Merge request reports

Loading