spacebar doesnt work flawless with handlers

[8bitbunny]

Bug
when i press on the spacebar on my keyboard with the miniplayer playing a video, 2 things happen: the page scrolls down, and the video pauses playing in miniview.

To Reproduce
as explained above

Expected behavior
expected behaviour to me would naturally be in this case: spacebar press to only pause/resume and not scroll the page. maybe use the page up/down keys for that specific?

System Configuration

• LBRY Daemon version: 0.63.0
• LBRY App version: 0.43.4
• LBRY Installation ID: not needed for this(?)
• Operating system: 64bit based Ubuntu 19.10 Linux. (KDE desktop environment)

Additional context
love LBRY alot! just wish i could help fixing the issues i come across :)

[tzarebczan]

Thanks for filing this issue, I can confirm it's happening. Can we show you some appreciation for the report?

[8bitbunny]

sure thing,
i will send an email from my domain i linked my lbry profile to.
which is not my github email, but it will be coming from 8bit-bun.nl
ramon :)

[harshkhandeparkar]

Is this browser specific? Didn't happen to me on Firefox.

[harshkhandeparkar]

I couldn't reproduce it on any browser or even the app. 😕

[cassidypignatello]

I've been able to reproduce this in both the app and on Chrome. It's my understanding that the default spacebar behavior in most browsers is to scroll down when the miniplayer isn't in focus. However, as soon as the miniplayer is brought back into focus it only pauses the video.

I think the best way to handle this would be to add an event listener that would disable the default behavior if a user presses spacebar. I'm not sure if there's any situation where you would want to keep this default behavior in place, so just thought I'd mention it just in case. Either way, I can start working on this within the next couple days.

[DispatchCommit]

Disabling spacebar seems like an overzealous solution. The player should just consume the spacebar input and stop the event from bubbling up so only 1 of the two actions occurs.

The solution in #4204 led to #4162 being opened, and goes against the standard behavior for the spacebar.