Fossil SCM

Blank cells in markdown tables are ignored

Fixed

95ce0e5302263e8… · opened 5 years, 7 months ago

Type
Code_Defect
Priority
Immediate
Severity
Important
Resolution
Fixed
Subsystem
Created
Aug. 19, 2020 1:13 a.m.

In a markdown table, blank cells are now ignored altogether and don't even work as placeholders. Consider the example table from https://fossil-scm.org/home/md_rules:

(markup)

| Header 1     | Header 2    | Header 3      |
----------------------------------------------
| Row 1 Col 1  | Row 1 Col 2 | Row 1 Col 3   |
|:Left-aligned |:Centered   :| Right-aligned:|
|              | ← Blank   → |               |
| Row 4 Col 1  | Row 4 Col 2 | Row 4 Col 3   |

(rendered)

| Header 1 | Header 2 | Header 3 |

| Row 1 Col 1 | Row 1 Col 2 | Row 1 Col 3 | |:Left-aligned |:Centered :| Right-aligned:| | | ← Blank → | | | Row 4 Col 1 | Row 4 Col 2 | Row 4 Col 3 |

That cell "← Blank →" used to appear in the middle column. Now it is on the left.

Comments (3)

andygoth 5 years, 7 months ago

In a markdown table, blank cells are now ignored altogether and don't even work as placeholders. Consider the example table from https://fossil-scm.org/home/md_rules:

(markup)

| Header 1     | Header 2    | Header 3      |
----------------------------------------------
| Row 1 Col 1  | Row 1 Col 2 | Row 1 Col 3   |
|:Left-aligned |:Centered   :| Right-aligned:|
|              | ← Blank   → |               |
| Row 4 Col 1  | Row 4 Col 2 | Row 4 Col 3   |

(rendered)

| Header 1 | Header 2 | Header 3 |

| Row 1 Col 1 | Row 1 Col 2 | Row 1 Col 3 | |:Left-aligned |:Centered :| Right-aligned:| | | ← Blank → | | | Row 4 Col 1 | Row 4 Col 2 | Row 4 Col 3 |

That cell "← Blank →" used to appear in the middle column. Now it is on the left.

Andy Goth 5 years, 7 months ago

Bisection reveals the problem was introduced by [2263cf086e9fabba96ce88451fa87cc32a1f0305].

Andy Goth 5 years, 7 months ago

Fixed by dc94ebc2.

Keyboard Shortcuts

Open search /
Next entry (timeline) j
Previous entry (timeline) k
Open focused entry Enter
Show this help ?
Toggle theme Top nav button