Skip to content

parse tbody and thead to markdown#35

Merged
stephenharris merged 1 commit intostephenharris:masterfrom
yilliot:master
Dec 7, 2013
Merged

parse tbody and thead to markdown#35
stephenharris merged 1 commit intostephenharris:masterfrom
yilliot:master

Conversation

@yilliot
Copy link
Copy Markdown
Contributor

@yilliot yilliot commented Nov 5, 2013

I found a bug which don't parse my table back with this sample of markdown.

| Year | Temperature (low) | Temperature (high) |  
| ---- | ----------------- | -------------------|  
| 1900 |               -10 |                 25 |  
| 1910 |               -15 |                 30 |  
| 1920 |               -10 |                 32 |  

so i did a tweak. now it work

@yilliot
Copy link
Copy Markdown
Contributor Author

yilliot commented Nov 5, 2013

I tried to implement doTables function for javascript in Markdown.Converter.js by referring js markdown extra

but my js skill not allow me to do that.

@stephenharris
Copy link
Copy Markdown
Owner

I've been following the progress of: https://github.com/jmcmanus/pagedown-extra

That's probably the most likely route for wp-markdown as it builds on the existing pagedown. The readme for pagedown-extra says the version of pagedown released in February 2013 is required (I can't remember off the top of my head what version wp-markdown is running).

I suspect, just updating pagedown and implementing pagedown-extra will be fairly straightforward...

(Initial tests of your patch have all passed :). Still creating more though....)

@stephenharris stephenharris merged commit 301506f into stephenharris:master Dec 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants