Improve Formatting Summary Text about bookshelf HOT 6 CLOSED

@mmcky, the summary is rendering with Markdown support. The reason why it looks like that is because the user supplied that Markdown. We could either disable Markdown (probably not what we want to do) or parse the summary text and remove any Markdown that we don't want to render.

from bookshelf.

Just noticed that we may need to parse the summary text and remove unwanted Markdown. Otherwise we end up with these kinds of things...

The bullet points, I think, are fine. But the big h1 header is pretty ugly

from bookshelf.

@tlyon3 Haha yep that's what I was going for.

The same can be said of comments; using H1 looks really silly.

from bookshelf.

Can you strip all HTML markup from the listing page summaries?
The summary should be a string of characters from the summary without any formatting.

Ideally the markdown shouldn't allow h1,h2,h3 etc. But that's a different issue.

@mmcky I'd push for this to be a A3 release issue (pretty important visually the homepage can break)?

from bookshelf.

@DrDrij I believe this tool (https://www.npmjs.com/package/sanitize-html) can help. I think it works for Markdown as well. I lets you whitelist allowed tags (strong, a, i, etc...) and removes the rest

from bookshelf.

I've blacklisted headers so they won't render. Lists, bold, italics, etc will still render. If there are other tags people want to be blacklisted, let me know.

Closing this issue because the headers were the main problem.

from bookshelf.