Nested tables

[L5tY1n]

Hello, I have some HTML with nested tables. Here is an example:

<table><tr><th>step</th><th>instructions</th></tr><tr><td>1</td><td><table><tr><th>condition</th><th>outcome</th></tr><tr><td>yes</td><td>approve</td></tr></table></td></tr></table>

When using ReverseMarkdown, it tries to create another Markdown table and uses <br> as newlines.
The Markdown parsers we use don't support this format.

I've looked around and apparently, the recommended way to fix this would be to output HTML.
The expected output would be

| step | instructions |
| --- | --- |
| 1 | <table><tr><th>condition</th><th>outcome</th></tr><tr><td>yes</td><td>approve</td></tr></table> |

Would it be possible to have the same mechanism as for ol or ul inside tables, and just return HTML for nested tables?

I tried to fix it manually using a child converter on my application to just return the HTML for nested tables, and it works on 4.8 with the example above, but I got into a bug in 4.7.1 because of the implementation of Tr.IsTableHeaderRow that sees the th in the nested table and decides we need to add an underline.

[mysticmind]

Acknowledge it, it is doable to not process the inner table. I will add a fix and do a release next week.

[laurent-yin]

Hello @mysticmind
Have you been able to look at this issue?

Thanks

[mysticmind]

@laurent-yin Hey, I have been traveling and fallen ill a bit. Sorry, will take a look sometime end of this week and keep you posted.

[L5tY1n]

Hello again @mysticmind,
I hope everything is well on your side.
Checking again on this issue, have you had time to look at it?

[mysticmind]

Hey Laurent, sorry, I had to rush on a family vacation, will certainly look at this on 31st once I am back in station.

[L5tY1n]

Thank you for fixing this, Babu!
What version will this fix be available in, do you have an idea when it will be available?

[mysticmind]

Thank you for fixing this, Babu! What version will this fix be available in, do you have an idea when it will be available?

@L5tY1n Planning to do sometime later today and it will be 5.0.0.

[mysticmind]

@L5tY1n Quick note, a big v5 release is done - https://github.com/mysticmind/reversemarkdown-net/releases/tag/5.0.0