Added <br/> and <seealso cref=xxx> #26
Conversation
…not already exist, create it first.
Pxtl
left a comment
Pxtl
left a comment
There was a problem hiding this comment.
Looks good. Some suggestions for improvement.
| } | ||
| ), | ||
| ["br"] = new TagRenderer( | ||
| "\n", |
There was a problem hiding this comment.
For proper markdown, shouldn't it be "\n " with two trailing space afterwards? Then it's compatible with standard-markdown in addition to github-flavored markdown.
https://stackoverflow.blog/2009/10/15/markdown-one-year-later/
| (x, context) => new string[0] | ||
| ), | ||
| ["seealso"] = new TagRenderer( | ||
| "##### See also: {0}\n", |
There was a problem hiding this comment.
If possible (and consider this just a suggestion):
https://gist.github.com/asabaylus/3071099
Github-flavoured markdown automatically generates an anchor for each heading. That means that seealso could be an intra-document link.
|
I've created my own fork and taken an initial stab at trying to fix anchor links and such: It also adds other things beyond the scope of this PR, like a few more HTML and Sandcastle/NDoc tags. |
3 participants
No description provided.