Add a space where two tags met

[danschumann]

I'm wondering if there is a way to create a space wherever tags were.

Picture this:

<div>Some sentence.</div><div>Some other Sentence</div>

It converts to Some sentence.Some other Sentence when I run

text = sanitizeHtml(text, {allowedTags:[], allowedAttribute: {}});

Is there an option to add whitespace so the output is better: Some sentence. Some other Sentence

[dgrad]

I think to do this properly you'll need a list of block tags (or a list of inline tags). You want to add a space wherever a block tag ends (actually it should probably be a newline character and let the browser convert it to space), but not where an inline tag ends (e.g. <div>foo</div><div>bar</div> should convert to foo bar, but <span>foo</span><span>bar</span> should convert to foobar).

[boutell]

I agree, and there should be an option to override that list.

I'd take a pull request for this one.

On Fri, Nov 27, 2015 at 12:42 PM, Daniel Grad notifications@github.com
wrote:

I think to do this properly you'll need a list of block tags (or a list of
inline tags). You want to add a space wherever a block tag ends (actually
it should probably be a newline character and let the browser convert it to
space), but not where an inline tag ends (e.g.

foo

bar

should convert to foo bar, but foobar should convert to foobar).

—
Reply to this email directly or view it on GitHub
#49 (comment)
.

*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE | (215) 755-1330 | punkave.com

[SystemDisc]

Until this gets implemented, a messy hack would be:

text = text.replace(/>/g, '> ');
text = sanitizeHtml(text, {allowedTags:[]});

[rafacustodio]

up!!

Is this still on?

[boutell]

@r-custodio As mentioned, I'd take a PR for this. Unfortunately as maintainer I can't necessarily implement every feature.

[greghub]

@abea this is labeled as seeking contributions but closed. Is it still something you'd accept a PR for?

[abea]

@greghub Sure. It had been sitting idle for years, so there didn't seem much reason to keep it open. I'll reopen it if you want to work on it for 2.x and let the stalebot close it if nothing happens.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[rusakovic]

Until this gets implemented, a messy hack would be:

text = text.replace(/>/g, '> ');
text = sanitizeHtml(text, {allowedTags:[]});

it's 2023 now. thank you for your solution =)

[boutell]

@rusakovic Contributions are welcome!

[adorum]

it's 2024 now. :)

[boutell]

Yes, it's 2024 now, and as always, community contributions are welcome. 😄 This isn't a feature that matters for our use cases, although I appreciate it would be nice for developers reading the resulting markup.

[boutell]

Reopening for potential community PRs.