Skip to content

Allow Inline HTML #1466

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

Closed
filiph opened this issue May 11, 2014 · 2 comments · Fixed by dart-archive/markdown#44
Closed

Allow Inline HTML #1466

filiph opened this issue May 11, 2014 · 2 comments · Fixed by dart-archive/markdown#44
Labels
package:markdown 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

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 changed the title Allow custom inline elements Allow Custom Inline Elements May 21, 2014
@filiph filiph unassigned dpeek Aug 3, 2015
@munificent munificent changed the title Allow Custom Inline Elements Allow Inline HTML Aug 5, 2015
@kevmoo kevmoo added the type-enhancement A request for a change that isn't a bug label Jun 14, 2017
@mosuem mosuem transferred this issue from dart-archive/markdown Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:markdown 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.

6 participants