Render GFM on snippet comments
What does this MR do and why?
Fixes Snippet comments don't appear to render front-e... (#582767 - closed) by correctly rendering GFM on "deprecated notes" loads (currently just snippet comments, I think), and ordering the existing renderGFM calls on note add/update correctly so they actually work.
The whole of deprecated_notes_spec.js has been quarantined for 5 years, which made testing this change hard. Many of the existing specs don't pass at all locally, so I've opted to drop them all into a new quarantined describe and unquarantine the top-level one (since its beforeEach and afterEach are useful), making it possible to add new specs again.
| Before | After |
|---|---|
![]() |
![]() |
How to set up and validate locally
-
Go make a snippet comment with a Mermaid diagram in it:
```mermaid graph TD; A-->B; A-->C; B-->D; C-->D; ``` -
Notice it's not rendered as it's added.
-
Refresh the page, and notice it's still not rendered.
-
Check out the branch.
-
Load the page. Notice it's rendered on load now!
-
Add a new comment with the diagram. Notice it's loaded as it's added, too.
MR acceptance checklist
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

