APP-6535: only fire clickOutside if click starts and ends outside

[mcous]

Overview

In browsers, if a click mousedown starts in one element and mouseup ends in another element, the click event is fired on the lowest common ancestor of the two elements.

If the button is pressed on one element and the pointer is moved outside the element before the button is released, the event is fired on the most specific ancestor element that contained both elements

MDN - click event

In most cases, this ancestor is going to trigger our clickOutside logic, resulting in frustrating user experiences, especially when selecting text.

Change log

Swap click listener for mousedown and mouseup listeners using the same "is target outside?" logic. If and only if both events are outside the target element, the handler is fired

[mcous]

Missing test

[mrloureed]

LGTM!