fix pipe tables (see #6) (#7)

Fix detection of separator lines:
1. Always check that at least one dash is present
2. Analyze whole content between pipes including white spaces

djot_parser/djot_ast.go

@@ -468,14 +468,18 @@ func buildDjotAst(
 				for s := i + 1; s < i+openToken.JumpToPair; s++ {
 					if list[s].Type == djot_tokenizer.PipeTableSeparator {
 						columns++
-						content := bytes.TrimSpace(document[list[s].End:list[s+list[s].JumpToPair].Start])
-						if bytes.HasPrefix(content, []byte(":-")) && bytes.Count(content, []byte("-")) == len(content)-1 {
+						content := document[list[s].Start+1 : list[s+list[s].JumpToPair].End-1]
+						dashCount := bytes.Count(content, []byte("-"))
+						if dashCount == 0 {
+							continue
+						}
+						if bytes.HasPrefix(content, []byte(":-")) && dashCount == len(content)-1 {
 							alignments = append(alignments, LeftAlignment)
-						} else if bytes.HasSuffix(content, []byte("-:")) && bytes.Count(content, []byte("-")) == len(content)-1 {
+						} else if bytes.HasSuffix(content, []byte("-:")) && dashCount == len(content)-1 {
 							alignments = append(alignments, RightAlignment)
-						} else if bytes.HasPrefix(content, []byte(":-")) && bytes.HasSuffix(content, []byte("-:")) && bytes.Count(content, []byte("-")) == len(content)-2 {
+						} else if bytes.HasPrefix(content, []byte(":-")) && bytes.HasSuffix(content, []byte("-:")) && dashCount == len(content)-2 {
 							alignments = append(alignments, CenterAlignment)
-						} else if bytes.Count(content, []byte("-")) == len(content) {
+						} else if dashCount == len(content) {
 							alignments = append(alignments, DefaultAlignment)
 						}
 					}

djot_parser/examples/complex_table.djot

@@ -0,0 +1,14 @@
+| XXX, XXXXX          | XXXXXX          | XXXXXX           |
+|---------------------|-----------------|------------------|
+| XXXXX XXXX          | XXXX-XX XXXX    | XXXXXXXXX, XX    |
+| XXXXX XXXXX (X)     | XX:XXXX         | X:XXXX           |
+|                     |                 |                  |
+| XXX, XXXXX          | XXXXXX          | XXXXXX           |
+|---------------------|-----------------|------------------|
+| XXXXX XXXX          | XXXXXXXXX, XX   | XXXXXX, XX       |
+| XXXXX XXXXXXX+ (X)  | X:XXXX          | XX:XXXX          |
+|                     |                 |                  |
+| XXX, XXXXX          | XXXXXX          | XXXXXX           |
+|---------------------|-----------------|------------------|
+| XXXXX XXXX          | XXXXXX, XX      | XXXX-XX XXXX     |
+| XXXXX XXXXX (X)     | X:XXXX          | X:XXXX           |

djot_parser/examples/complex_table.html

@@ -0,0 +1,57 @@
+<table>
+<tr>
+<th>XXX, XXXXX</th>
+<th>XXXXXX</th>
+<th>XXXXXX</th>
+</tr>
+<tr>
+<td>XXXXX XXXX</td>
+<td>XXXX-XX XXXX</td>
+<td>XXXXXXXXX, XX</td>
+</tr>
+<tr>
+<td>XXXXX XXXXX (X)</td>
+<td>XX:XXXX</td>
+<td>X:XXXX</td>
+</tr>
+<tr>
+<td></td>
+<td></td>
+<td></td>
+</tr>
+<tr>
+<th>XXX, XXXXX</th>
+<th>XXXXXX</th>
+<th>XXXXXX</th>
+</tr>
+<tr>
+<td>XXXXX XXXX</td>
+<td>XXXXXXXXX, XX</td>
+<td>XXXXXX, XX</td>
+</tr>
+<tr>
+<td>XXXXX XXXXXXX+ (X)</td>
+<td>X:XXXX</td>
+<td>XX:XXXX</td>
+</tr>
+<tr>
+<td></td>
+<td></td>
+<td></td>
+</tr>
+<tr>
+<th>XXX, XXXXX</th>
+<th>XXXXXX</th>
+<th>XXXXXX</th>
+</tr>
+<tr>
+<td>XXXXX XXXX</td>
+<td>XXXXXX, XX</td>
+<td>XXXX-XX XXXX</td>
+</tr>
+<tr>
+<td>XXXXX XXXXX (X)</td>
+<td>X:XXXX</td>
+<td>X:XXXX</td>
+</tr>
+</table>

djot_parser/examples/complex_table2.djot

@@ -0,0 +1,3 @@
+| -- | -- | -- |
+|--|--|--|
+| - | -- | ---- |

djot_parser/examples/complex_table2.html

@@ -0,0 +1,12 @@
+<table>
+<tr>
+<th>&ndash;</th>
+<th>&ndash;</th>
+<th>&ndash;</th>
+</tr>
+<tr>
+<td>-</td>
+<td>&ndash;</td>
+<td>&ndash;&ndash;</td>
+</tr>
+</table>