Fix bug #19 where the extension would fail to format a table with col…

…umns header and lines with 1 character
fcrespo82 · Jan 24, 2020 · a063408 · a063408
1 parent 18e33c0
commit a063408
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,11 @@
+## 2.0.3
+
+* [FIX] Fix bug #19 where the extension would fail to format a table with columns header and lines with 1 character
+
+## 2.0.2
+
+* [CHANGED] A lot of internal fixes and improvements 
+
 ## 2.0.1
 
 * [FIX] Various bugs

diff --git a/src/MarkdownTableUtils.ts b/src/MarkdownTableUtils.ts
@@ -13,7 +13,9 @@ export let swidth = (str: string) => {
 };
 
 export let padding = (len: number, str: string = ' ') => {
-	return str.repeat(len);
+	var r = len >= 0 ? str.repeat(len) : "";
+	// var r = str.repeat(len);
+	return r;
 };
 
 export let columnSizes = (header: string[], body: string[][]) => {

diff --git a/src/formatter/MarkdownTableFormatterProvider.ts b/src/formatter/MarkdownTableFormatterProvider.ts
@@ -186,7 +186,7 @@ export class MarkdownTableFormatterProvider implements vscode.DocumentFormatting
 						line = front + padding(table.columnSizes[i] + (settings.spacePadding * 2) - 2, '-') + back;
 						break;
 					case MarkdownTableFormatterDelimiterRowPadding.FollowSpacePadding:
-						line = `${spacePadding}${front}${padding(table.columnSizes[i] - 2, '-')}${back}${spacePadding}`;
+						line = `${spacePadding}${front}${padding(table.columnSizes[i] - 2, '-')}${table.columnSizes[i]===1?'':back}${spacePadding}`;
 						break;
 					case MarkdownTableFormatterDelimiterRowPadding.SingleApaceAlways:
 						line = ` ${front}${padding(table.columnSizes[i] + (settings.spacePadding * 2) - 4, '-')}${back} `;