sirthias / pegdown

A pure-Java Markdown processor based on a parboiled PEG parser supporting a number of extensions
http://pegdown.org
Apache License 2.0
1.29k stars 218 forks source link

Fix "Error: <aside> is not recognized!" and alike #164

Closed martinklepsch closed 9 years ago

martinklepsch commented 9 years ago

Actually the current change does not seem to fix this problem but I'm sort of out of ideas. I get messages like Error: <aside> is not recognized! for various tags like <figure>, <figcaption> and many more.

I initially thought that modifying HTML_TAGSshould do the trick but could not observe any change. I was searching the repo for strings like not recognized to figure out where the warning comes from but couldn't find any.

Any guidance appreciated. I hope we all agree that even if it's markdown people should generally be allowed to use all HTML tags.

Deraen commented 9 years ago

The problem is in Endophile (Clojure wrapper library).

Deraen commented 9 years ago

Though the mentioned error comes from another library this change is still required to recognize figure and others tags as html blocks. But this commit is broken as HTML_TAGS is searched using binary search and thus the tags must be be ordered.

martinklepsch commented 9 years ago

Closing this in favor of #169