Escape non breaking space

[predragnikolic]

This should address this comment #1428 (comment)

This is a try I am not sure if this works.
I do not know how to test this manually locally. I run the tests npm run test, and the test are passing so that makes me think that everthing should work...

So feel free to push some commits.

Checklist

• only relevant code is changed (make a diff before you submit the PR)
• run tests npm run test
• tests are included

Checklist (for documentation change)

• only relevant documentation part is changed (make a diff before you submit the PR)
• motivation/reason is provided

[adrai]

@cuyl can you review this?

[coveralls]

Coverage remained the same at 96.057% when pulling e7cc438 on predragnikolic:escape-non-breaking-space into 6fda214 on i18next:master.

[cuyl]

@adrai   is works without any changes.

@predragnikolic You can revert react-i18next.js change and run test, it works.

[predragnikolic]

  is works without any changes.

Are you sure?

I have the following:

<Trans
    shouldUnescape
    defaults={"<p>Do&nbsp;Not&nbsp;Disturb</p>"} />

And I expect to see:

But I see:

[adrai]

@predragnikolic this works without any change: 71d3542

[predragnikolic]

this works without any change:

True the test passes even without my changes,
but I am confused why the browser still displays &nbsp;

[cuyl]

@predragnikolic If you want render many actual spaces, you can try \u00A0 or \xA0. unescape   to SPACE can't help you, because even you unescape   to SPACE, html still only render one space.

[predragnikolic]

I'd say that I expected the wrong behavior, so I'm closing this. :)