search

yogthos / markdown-clj

Markdown parser in Clojure
Eclipse Public License 1.0
540 stars 120 forks source link

Tables with less spaces #164

Closed rutchkiwi closed 4 years ago

rutchkiwi commented 4 years ago

Like these ones:

| apa | banan |
|-----|-------|
| 1   | 2     |

Before this change, only this syntax was allowed:

| apa | banan |
| --- | ----- |
| 1   | 2     |

which is a bit of a bummer since awesome tools like https://stevecat.net/table-magic/ gives me the no-spaces syntax.

At my job the content team is responsible for creating these sort of markdown tables for product ingredients, and they ALWAYS mess it up so I thought they'd appricate some automated tools and a more leinient parser.

Oh, and thanks for this awesome library :)

yogthos commented 4 years ago

Totally makes sense, thanks for the fix!

yogthos commented 4 years ago

and up on Clojars

rutchkiwi commented 4 years ago

Thanks a lot! :)

rutchkiwi commented 4 years ago

Hey I upgraded the clojars version just fine now, but how do I get the JS version? I seem to remember grabbing the JS file from the demo page but that does not seem to be updated?

Edit: Actually now I realized that I should probably have ran cljsbuild in the PR to update the demo file. But still not sure how it goes into NPM (I don't use NPM so don't care too much - maybe it is already there?)

yogthos commented 4 years ago

Oh right, I have to push out a new version to NPM separately. Let me take a look here. :)

yogthos commented 4 years ago

ok latest should be up on npm