continue past alert and confirm dialogs with keyboard space or enter

here's a simple patch just adds the "autofocus" attribute the dialogs' primary button.

https://caniuse.com/#search=autofocus says this should work for chrome and firefox, but not iOS Safari.

I haven't tested this, but I'm not in favor of the 'enter' key having the ability to dismiss dialogs.

I'd be in favor of using the spacebar to proceed, but barcode scanners are generally programmed to add an 'enter' at the end of a scan, which This could result in:

A. Users missing the pop-up alert due to a subsequent barcode scan, and also

B. Missing the barcode that dismissed the alert.

I agree with Michele.

thank you for the feedback, i hadn't considered that.

here's a different patch that proceeds/confirms/continues with space only.

a button can still be activated if you press enter while the button has focus.
(that is: if you press tab to get to the button, then press enter)

This patch arose from a discussion in the Circulation Interest Group at this year's conference. I asked if anyone knew what keystroke would work on dismissing such messages because we had not found one and were having to mouse it. No one else at the discussion knew of one, perhaps not enough people there had transitioned on to webby to encounter it.

Thankfully, abellenir attended that session and was able to find the ESC key worked and he asked what keystrokes we had used in the past to handle them. In my case at circulation, the space bar and he was able to offer up a potential fix for it.

Thanks again for your interest in end-user accessibility issues abellenir!

+1 to allowing space bar to close dialogs in addition to Esc key.

Adding abellenir's patch to a working branch.

Also implementing the same logic in the Angular alert/confirm dialogs, which are (too easily) closed with a simple Enter key.

Working branch en route.

Branch w/ 2 commits as promised:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1826584-space-closes-some-dialogs

Note the Angular commit includes sandbox examples.

Non-Codabar barcodes can have spaces in them. This will be fun!

I wonder how common it is for item barcodes that get checked out across the desk to legitimately have spaces in them.

But if we have to accommodate that possibility, perhaps we could consider an org unit setting to allow continuing past dialogs with the space bar.

Or, since barcodes that conform to the strict barcode algorithm can't have spaces, can we somehow tie the confirm with space behavior to the strict barcode workstation setting?

Adding a link to a recent IRC discussion:

http://irc.evergreen-ils.org/evergreen/2019-09-04#i_417423

I also added the accessibility tag, there needs to be a way to continue past dialogs using the keyboard.

Removing pullrequest and adding needsdiscussion as per comments #8 and 9.