Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

parse tbody and thead to markdown #35

Merged
merged 1 commit into from
Dec 7, 2013
Merged

parse tbody and thead to markdown #35

merged 1 commit into from
Dec 7, 2013

Conversation

yilliot
Copy link
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
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
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.

None yet

2 participants