fix F-keys issue mentioned in #41

Ademking / BetterViewer

a replacement for the image viewing mode built into Firefox and Chrome-based web browsers.

MIT License

252 stars 19 forks source link

fix F-keys issue mentioned in #41 #42

Closed Metacor closed 2 years ago

Metacor commented 2 years ago

This... was a very simple fix, I honestly don't know why I didn't do it this way in the first place.

I added an e parameter to the Mousetrap.bind function, then I just moved e.preventDefault(); within it, instead of doing the weird multi-key combination comparison

Ademking commented 2 years ago

LGTM! Thank you @Metacor ! You're the best ❤️