wiki formatting problem: tables are not indented

[vittorio]

tables arent indented when starting the lines with a few spaces. for example:

  ||A||B||

    ||C||D||

A

B

C

D

following html code is generated:

<blockquote>
</blockquote>
<table class="wiki">
<tr></td></tr></table>

<p>
<td>A</td><td>B
</p>
<blockquote>
<blockquote>
</blockquote>
</blockquote>
<table class="wiki">
<tr></td></tr></table>
<p>
<td>C</td><td>D
</p>

using trunk r790

[Christian Boos]

Please, don't close a bug without a comment stating why you closed it.

By the way, the generated HTML with the current trunk is still the same as the one reported, so the problem (if there is a problem) remains.

[Christopher Lenz]

Postponing again.

[Christopher Lenz]

And again.

[Christopher Lenz]

And again.

[Alec Thomas]

Seems to work, all unit tests pass.

[Alec Thomas]

Patch against r3234 with minor fix for paragraph breaks

[Christian Boos]

Alec, I've tested the patch and it works well. I think you can commit it.

[Alec Thomas]

I found a bug unfortunately :( If tables are separated by newlines I can make it work, but if they are like this:

||a||
  ||b||
||c||

The third table is joined with the second. Anyway, I'll keep playing until I figure it out :)

[Christian Boos]

Yes, I've seen that too, but it didn't bothered me that much: it's something that the user can easily "fix" by adding an extra newline, whereas the other problems your patch fixes can't be fixed by the user. So I think that applying the patch and keeping note of the above issue is better that leaving the issue open waiting for the "perfect" fix (which will only be perfect until someone find another edge case anyway :) ).

[Alec Thomas]

Okay, I think I've hit a good compromise in r3261. Indented tables work 100% correctly, except that rows can no longer be separated by blank lines.