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

Just to put this on your radar #712

Closed
wants to merge 5 commits into from
Closed

Conversation

nickevansuk
Copy link
Contributor

Hi Marcos,

Just to let you know that the new version of the marked library includes table support for markdown, which you can activate if you switch to gfm mode. It also works quite nicely with highlight.js which I've also added for markdown mode (though understand you already have google-code-prettify in there so maybe don't want to conflate the two for the W3C usecase).

Anyway here's the code - let me know if you want me to cherry pick some of the changes into a proper pull request for you. I still need to fix the unit tests to work with gfm so it's not ready as-is.

We're a new W3C CG and we're using Respec for our specs - so will continue to pull request changes we make in case they're useful. Let me know if this is of interest.

Great work on Respec by the way! Robin's left a solid legacy with this, so well done for carrying the torch!

Thanks,

Nick

@marcoscaceres
Copy link
Member

@nickevansuk, this is great to see. I had filed a bug to do this (#581), but ran into difficulties on my first attempt - so it's great to see you got it working.

Would love to coordinate on upgrading marked.js and the highlighter.

@marcoscaceres
Copy link
Member

Closing this for now - but looking forward to collaborating.

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