It adds extra
<em> after the comma.
Not great to have
, inside links indeed, but I perhaps this could be fixed anyhow?
Spaces are not allowed in URLs, they are supposed to be URL-encoded. Commas should be URL-encoded as well. How did you obtain a URL with embedded commas and spaces?
Both commas and underscores (which I think is the problematic character here) are perfectly fine in URLs. The comma is a reserved character so may be percent escaped to avoid it being interpreted in a reserved manner by the server, but the client shouldn’t care either way.
In this case it looks like the markdown parser is looking too deep into the tag…
And how is the URL supposed to be
Logic is in a nasty regex that I do not want to hack on, we get a brand new regex with the new markdown engine, this fix will have to wait till then
This now works, for added bonus it also auto links correctly.
migration plans, I just enabled markdown.it at Meta.
It is an excellent CommonMark compliant engine. I expect quite a few edges to be found during this period of testing.
Do not open bugs about the new Engine, just reply here.
I/we will be updating the top post here with the list of issues and striking out stuff as it is fixed.
Current known issues (to be fixed)
Onebox vanishes after you start typing and works not-super consistently