TableFormat does not remove colons for alignment

Fix #265, #220
preservim · Feb 25, 2016 · e07c336 · e07c336
1 parent 680f0dc
commit e07c336
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/ftplugin/markdown.vim b/ftplugin/markdown.vim
@@ -459,8 +459,11 @@ function! s:TableFormat()
     " Remove everything that is not a pipe othewise well formated tables would grow
     " because of addition of 2 spaces on the separator line by Tabularize /|.
     let l:flags = (&gdefault ? '' : 'g')
-    execute 's/[^|]//' . l:flags
+    execute 's/\(:\@<!-:\@!\|[^|:-]\)//e' . l:flags
+    execute 's/--/-/e' . l:flags
     Tabularize /|
+    execute 's/:\( \+\)|/\1:|/e' . l:flags
+    execute 's/|\( \+\):/|:\1/e' . l:flags
     execute 's/ /-/' . l:flags
     call setpos('.', l:pos)
 endfunction

diff --git a/test/table-format.vader b/test/table-format.vader
@@ -29,3 +29,16 @@ Expect (table is not modified):
   | a | b |
   |---|---|
   | c | d |
+
+Given markdown;
+| left |right|  center  ||
+| :- | --: |:---:|:|
+| left |right|  center  ||
+
+Execute (format table with colons):
+  TableFormat
+
+Expect (preserve colons to align text):
+  | left | right | center |   |
+  |:-----|------:|:------:|:--|
+  | left | right | center |   |