continue past alert and confirm dialogs with keyboard space or enter

[a. bellenir]

it would be convenient if a user could proceed past an alert or confirm dialog by pressing 'enter' or 'space' on the keyboard.

(the 'escape' key can already be used to dismiss dialogs)

[a. bellenir (abellenir)]

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.

[Michele Morgan (mmorgan)]

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.

[Tiffany Little (tslittle)]

I agree with Michele.

[a. bellenir (abellenir)]

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)

[Jennifer Bruch (jbruch)]

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!

[Bill Erickson (berick)]

+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.

[Bill Erickson (berick)]

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.

[Jason Stephenson (jstephenson)]

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

[Michele Morgan (mmorgan)]

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.

[Terran McCanna (tmccanna)]

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

[Stephanie Leary (stephanieleary)]

I've put a branch on bug 1947868 that also addresses this bug.