Newline handling

[obecker]

If the text to be clamped contains line breaks, then they will be preserved in the created element (by adding <br> elements). This should not happen. The display of line breaks should only be controlled by the CSS white-space property.

Solution: the text should be inserted by using textContent instead of innerText.
See https://developer.mozilla.org/en-US/docs/Web/API/Node/textContent

Moreover, if line breaks are preserved and the last displayed line is short enough for the containing element, it is still shortened before adding the ellipsis.
Example:

This is some example text
with multiple
short lines
to demonstrate
the behaviour.

Using <NanoClamp text={text} lines={2} /> for this text (in a wide enough div) results in

This is some example text
with mult...

I would expect that the word multiple is not shortened (since it's not necessary), so the result should rather be

This is some example text
with multiple...

Solution: in the loop that finds the proper index in the text always append the ellipsis before checking the height of the element.

[obecker]

See #29