Added missing previous selection discard upon the new selection change

[oleksandr-danylchenko]

Issue

Initially reported - #151 (comment)

We need to restore the selection.clear() call if there's nothing to update. Because now the previous, "discarded selection" unexpectedly stays up until you create a brand-new one.

unselect.mp4

*in my testing env, the annotations that don't have either a highlight or a note assigned get automatically garbage collected when they get unselected

Can be reproduced in the playground:

selected_still.mp4

[oleksandr-danylchenko]

The isEmpty check is redundant because such a case is already handled on the a9s side. The operation will be no-op if the selection is already clear. Source

[rsimon]

Thanks - yes, that makes sense. The previous selection should clear as soon as you start the next, not after you complete it.

[oleksandr-danylchenko]

Another mobile quirk!
On iOS the ordering is the same as on Android - the selection comes after the pointer events. BUTT! It fluctuates depending on where and how fast you click!
When you click over an annotation quickly - only pointerdown/pointerup are dispatched. But when slowly - all 4. When I tap over the textual content - all 4.

Screen.Recording.2024-10-16.at.21.33.21.mov