preventDefault for F5 - F8 and Backspace? about diabloweb HOT 5 CLOSED

I'm not a fan of blocking all keyboard shortcuts but maybe this is the right time to do it. I'll put it in and see if it causes any issues.

from diabloweb.

Okay, preventing default on all keypresses was a disaster. Limiting it to Backspace and F1/F5 for now.

What is the purpose for mapping F5-F8 to numpad?

from diabloweb.

oh, didn't realize you prevent default on all keypresses. I wrote a tampermonkey script to prevent it only for f1, f5, and backspace and haven't have any issue.

F5-F8 were used as hotkey to switch skills. As you would suspect, only f6 and f8 work flawlessly, f5 will refresh the page and f7 will call out a firefox window.

from diabloweb.

I tried using KeyboardEvent to remap F5-F8 w/o any success. I suspect it's not allowed for security reasons (see quote from MDN below). Besides, even if it works, it would still trigger as Browser's shortcut. Is there a way to change the key binding inside complied devilution wasm files? Numpad 1 - 4 will work well for this.

Manually firing an event does not generate the default action associated with that event. For example, manually firing a key event does not cause that letter to appear in a focused text input. In the case of UI events, this is important for security reasons, as it prevents scripts from simulating user actions that interact with the browser itself.

from diabloweb.

Ah, right. Thought you meant mapping the other way. Extended it to F1-F8 plus backspace.

from diabloweb.