don't use mousedown events instead of click

[Freeplayg]

First thing, this is not expected behavior. The action should only be done once the left mouse button has been lifted

Second, it breaks a lot of behavior. You can't drag, right click, middle click, navigational links without it triggering. This makes things more frustrating if you try to do something like opening a page in a new tab.

[samolukadjo]

Oh yeah, I second this.

[FireIsGood]

You'll have to bring it to @gabboman. I also suggested this in the dev Discord but Gabbo noted that it was faster and linked this video. Sharing this to provide some context.

Gabbo feel free to clarify anything I've stated here, this is interpretation of what we discussed last week.

[Freeplayg]

I haven't fully watched the linked video but even just reading through the comments, it's a big accessibility issue using mousedown instead of click.

It shouldn't be something used everywhere

[FireIsGood]

I'll also note after testing that pressing enter on an element with tabIndex will trigger the (click) event but not the (mousedown) event. This is why I had to implement custom (keydown) event handling for the navigation in #244.