search

hey-red / Markdown

Open source C# implementation of Markdown processor, as featured on Stack Overflow.
MIT License
134 stars 51 forks source link

Compatibility issues with MarkdownSharp #77

Closed matt-richardson closed 8 years ago

matt-richardson commented 8 years ago

There are a few other compatibility issues, but not as impacting as #75 . 1) Transform in the MarkdownSharp code base returns the string with a trailing \n, whereas this one does not. 2) AllowEmptyLinkText defaults to false whereas I think it needs to default to true to keep matching behaviour.

hey-red commented 8 years ago

1). I cant find nothing about this in md spec. There are good reasons to add it ? 2). It makes sense (but I do not like that you can create a hidden link). Create PR or I'll do it but later.

matt-richardson commented 8 years ago

I'm more thinking more from a in-place replacement point of view rather than whether its a good idea or not :)

If i get a chance I'll create the PR, but as there's a workaround its not as high on the priority list.

hey-red commented 8 years ago

@matt-richardson AllowEmptyLinkText currently = true 61e1fa6