#853@minor: Fix method Range.cloneContents

[danielrentz]

fix #853

Seems the method initializes the initial subrange (the first element containing the start of the selection) incorrectly (Range.ts, line 282):

• Currently, the subrange will be created from end of current range to end of firstPartialContainedChild.
• Obviously, the subrange should clone from start of original range to end of firstPartialContainedChild.

Seems that existing unit test did not catch this because selection does not start inside a sub element.

Minor additional fix: I removed the else-if branches in the startOffset and endOffset getters. Code would only run into the else-if branch if the offset property is already 0 (otherwise the initial if-branch would trigger and reset the offset property to 0). Therefore the else-if branch does not have any effect.

[capricorn86]

NIce find! 🌟