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

Allow Inline HTML #18

Closed
filiph opened this issue May 11, 2014 · 2 comments · Fixed by #44
Closed

Allow Inline HTML #18

filiph opened this issue May 11, 2014 · 2 comments · Fixed by #44
Labels
type-enhancement A request for a change that isn't a bug

Comments

@filiph
Copy link
Contributor

filiph commented May 11, 2014

Not sure if this is in scope, but markdown normally allows HTML tags. You can write something like this:

This is **markdown'd**. But you can also <span color="red">do stuff manually</span>.

The span should be copied verbatim to the HTML output. In this implementation, though, all HTML tags are escaped (< becomes &lt).

@dpeek
Copy link
Contributor

dpeek commented May 19, 2014

This is supported in the block parser, but not inline:
https://github.com/dpeek/dart-markdown/blob/master/lib/src/block_parser.dart#L325

I'll take a look at it.

@0xcaff
Copy link
Contributor

0xcaff commented May 20, 2014

@dpeek Since dart is primarily for the web, we should add a safeHtml option to this library.

@0xcaff 0xcaff added the bug label May 20, 2014
@0xcaff 0xcaff changed the title Allow custom inline elements Allow Custom Inline Elements May 21, 2014
@filiph filiph unassigned dpeek Aug 3, 2015
@munificent munificent added enhancement and removed bug labels Aug 5, 2015
@munificent munificent changed the title Allow Custom Inline Elements Allow Inline HTML Aug 5, 2015
@kevmoo kevmoo added type-enhancement A request for a change that isn't a bug and removed enhancement labels Jun 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants