Tables

[redsun82]

Hi, I implemented simple tables for the maruku dialect (more or less following PHP Markdown Extra's implementation), as I needed them for a project of mine where I use this lib. Tests are still missing though...

A difference is that this implementation does not force cells in a row to be created up to the maximum width of the table... don't know yet if it's a big deal.

There are also some other minor changes that ended up in the code: the correction of a leaking global variable, the parameterisation of escaped characters per dialect (markdown extra apparently needs escaping for : and |), and a break statetement to exit a loop cheking for an empty dictionary.

edit: corrected typo

[ashb]

Thanks for this.

Could we have some tests that cover at least a few common cases of tables even if they are not exhaustive?

[redsun82]

Sure, I'll try to have them up by today or tomorrow.

2012/10/31 Ash Berlin notifications@github.com

Thanks for this.

Could we have some tests that cover at least a few common cases of tables
even if they are not exhaustive?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/66#issuecomment-9946506.

[redsun82]

Here they are.

[vanthome]

Why hasn't this been merged yet?

[ashb]

Two reasons, the main one being I'd forgotten about it - sorry. The second is that the change to how escapes are checked doesn't work (i.e. the tests don't pass) - I'm looking at this now.

[vanthome]

Great, will test as soon as you release this

[ashb]

@vanthome Just tidying up a few things and aiming to fix one more thing then a 0.5 will be on its way to npm

[ashb]

@vanthome v0.5.0 just published to NPM.

@redsun82 Thanks for this - sorry it took so long to merge.

[vanthome]

Hmm, are you sure that you pushed the latest version to GH?,
I'm using this version and it does not seem to work :(

[ashb]

You'll need to make sure you select the Maruku dialect - the default mode of operation is compatible with Gruber's vanilla script.

[vanthome]

Ahh ok, I document this, but it still won't work:

I'm using it in the browser and doing this:

  markdown.Markdown('Maruku');
  html = markdown.toHTML(data);

And I took my table from your test data.

[ashb]

Do you get an error or it just doesn't produce the output you expected?

[vanthome]

no error, just the tables are rendered plain

[ashb]

Oh I see. markdown.Markdown('Maruku') creates an instance with the dialect set, but you've thrown that away.

html = markdown.toHTML(data, "Maruku");

should work for you.

[vanthome]

I tried this actually before but try to do this in the browser console of a page where markdown-js is included:

markdown.Markdown('Maruku');

you will see that it returns undefined

[evilstreak]

You need to use new markdown.Markdown('Maruku');

[vanthome]

I think this does not work in the browser.
When I try this in console, I receive a useless object:

var test = new markdown.Markdown('Maruku');
Object.keys(test);
["dialect", "em_state", "strong_state", "debug_indent"]

[evilstreak]

Sorry, my comment was a bit hasty and not all that helpful.

The exposed parse function uses this approach. It's really only necessary if you want to muck around with the intermediate trees, rather than going from a Markdown string to an HTML string.

If you just want to go from string to string with a specific dialect, then as @ashb said you just want to pass the dialect into your toHTML call.

[vanthome]

This works, excellent, thx!