Heading misalignment with markdown rendering in notebook

[caohai]

Version: 1.36.0 insider

Steps to Reproduce:

1. Open a new notebook
2. Paste something like this into a text cell with Markdown view:

## Sub1
- a
## Sub2
## Sub3

Notice that Sub2 and Sub3 are not correctly aligned.

The same md doc can be correctly rendered by VSCode and Github, example:

Sub1

• a

Sub2

Sub3

PS: This can be worked around by adding an extra new line before the heading.

[chlafreniere]

This is due to an issue in marked.js (the md renderer that ADS uses). Issue looks like the same root cause as markedjs/marked#1047.

Since this isn't a new issue, has an easy workaround (just adding a newline), and is in one of our dependencies, moving this out from April, but keeping tentatively in June.

Here's a demo of marked.js not handline this properly: https://marked.js.org/demo/?text=%23%23%20Sub1%0A-%20a%0A%23%23%20Sub2%0A%23%23%20Sub3&options=%7B%0A%20%22baseUrl%22%3A%20null%2C%0A%20%22breaks%22%3A%20false%2C%0A%20%22extensions%22%3A%20null%2C%0A%20%22gfm%22%3A%20true%2C%0A%20%22headerIds%22%3A%20true%2C%0A%20%22headerPrefix%22%3A%20%22%22%2C%0A%20%22highlight%22%3A%20null%2C%0A%20%22langPrefix%22%3A%20%22language-%22%2C%0A%20%22mangle%22%3A%20true%2C%0A%20%22pedantic%22%3A%20false%2C%0A%20%22sanitize%22%3A%20false%2C%0A%20%22sanitizer%22%3A%20null%2C%0A%20%22silent%22%3A%20false%2C%0A%20%22smartLists%22%3A%20false%2C%0A%20%22smartypants%22%3A%20false%2C%0A%20%22tokenizer%22%3A%20null%2C%0A%20%22walkTokens%22%3A%20null%2C%0A%20%22xhtml%22%3A%20false%0A%7D&version=master

[chlafreniere]

This has been fixed in markedjs/marked#2483, and released as marked.js 4.0.17.

This will eventually make its way into our codebase with vscode merges. Unfortunately, the version of marked.js is too old in the ADS codebase, so I can't just port this one fix without a decent amount of rework.