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

Disabled global HTML escaping, while always escaping within code blocks. #87

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

michaek
Copy link

@michaek michaek commented May 2, 2013

This follows from #34, opting to disable HTML escaping except in code blocks. I understand that it's been stated to be undesirable by the maintainers, but I think it is a more consistent implementation of Markdown.

I've added a test that mimics the handling here http://babelmark.bobtfish.net/?markdown=%3Cp%3EHTML%3C%2Fp%3E%0D%0A%0D%0A++++%3Cp%3ECode%3C%2Fp%3E&src=1&dest=2

@michaek
Copy link
Author

michaek commented Jun 18, 2013

Any feedback? I think the Babelmark link makes it pretty clear that escaping HTML outside a code block is unconventional for Markdown. (Not to mention that the Markdown documentation is explicit about allowing HTML.)

@ozten
Copy link

ozten commented Jul 1, 2013

+1

I also need [an optional flag for] markdown-js to pass through HTML markup which is in Markdown files.
I forked and cherry-picked michaek's work. WFM so far.

@ozten
Copy link

ozten commented Jul 2, 2013

I found a bug with this patch, noted here.

@nunull
Copy link

nunull commented Jan 30, 2014

Anything new regarding HTML-support?

@dustMason
Copy link

I'm also feeling the need for this option. Any progress here?

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.

4 participants