Bug 21346 made some dialogs in returns.pl modal. Some libraries think this is too distruptive for their workflow and would like to continue scanning books even if there is a dialog/message.
Perhaps we could use humanmsg with dismissal on keyinput/scan?
Created attachment 90539 [details] [review] Bug 23007: Make modals close on enter
Just an example, we could make the transfer modals close on enter, we could tie it to a pref if needed
Some libraries don't want staff scanning through, because they ignore the message. If you are going to make it scan through, it should be something that can be turned on or off.
Hi all, The change in having to say okay each time an item transfers is time consuming for my staff. We only have one branch, so we always where it is coming from or going to. We would love to see a patch soon. Thanks for all the things everyone does for the community here in bugland. Caroline Ford
I agree with Christopher. We have 51 libraries and changing the transfer message to a modal is a good thing for us. Since it's not a good thing for everyone, though, I think whether this message can be scanned through or not should be a system preference. George
Created attachment 91049 [details] [review] Bug 23007: Add TransfersBlockCirc system preference
Created attachment 91050 [details] [review] Bug 23007: Retain focus on 'barcode' input if transfers don't block circ To test: 1 - Apply patches 2 - Update database 3 - Confirm TransfersBlockCirc defaults to 'Block' 4 - Check in an item that needs transfer 5 - Confirm you must deal with the modal to proceed 6 - Repeat checking to get 'wrong-transfer-modal' 7 - Confirm you must deal with it to proceed 8 - Switch the pref to 'Don't block' 9 - Find a new item or cancel/fulfill the preceeding transfer 10 - Check in the item to generate transfer 11 - Note focus remains on the barcode field and you can continue scanning (or typing if you don't have a scanner) 12 - Note typing and pressing enter continues without requiring you to deal with modal 13 - Repeat for wrong transfer modal 14 - Verify hold and other modals still block continuing circulation
Please make this a system preference. We choose not to interrupt workflow.
Should it be assumed that if you are not blocking, you probably are not printing, and therefore, the modal doesn't need to appear at all? Or if so, what about just putting the print slip button on the new line in the form with the title instead of having the modal? I'm just thinking that the modal covers things, and if people are not going to care about it, why show it?
Created attachment 91506 [details] [review] Bug 23007: Add TransfersBlockCirc system preference Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Created attachment 91507 [details] [review] Bug 23007: Retain focus on 'barcode' input if transfers don't block circ To test: 1 - Apply patches 2 - Update database 3 - Confirm TransfersBlockCirc defaults to 'Block' 4 - Check in an item that needs transfer 5 - Confirm you must deal with the modal to proceed 6 - Repeat checking to get 'wrong-transfer-modal' 7 - Confirm you must deal with it to proceed 8 - Switch the pref to 'Don't block' 9 - Find a new item or cancel/fulfill the preceeding transfer 10 - Check in the item to generate transfer 11 - Note focus remains on the barcode field and you can continue scanning (or typing if you don't have a scanner) 12 - Note typing and pressing enter continues without requiring you to deal with modal 13 - Repeat for wrong transfer modal 14 - Verify hold and other modals still block continuing circulation Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Created attachment 92506 [details] [review] Bug 23007: Add TransfersBlockCirc system preference Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Created attachment 92507 [details] [review] Bug 23007: Retain focus on 'barcode' input if transfers don't block circ To test: 1 - Apply patches 2 - Update database 3 - Confirm TransfersBlockCirc defaults to 'Block' 4 - Check in an item that needs transfer 5 - Confirm you must deal with the modal to proceed 6 - Repeat checking to get 'wrong-transfer-modal' 7 - Confirm you must deal with it to proceed 8 - Switch the pref to 'Don't block' 9 - Find a new item or cancel/fulfill the preceeding transfer 10 - Check in the item to generate transfer 11 - Note focus remains on the barcode field and you can continue scanning (or typing if you don't have a scanner) 12 - Note typing and pressing enter continues without requiring you to deal with modal 13 - Repeat for wrong transfer modal 14 - Verify hold and other modals still block continuing circulation Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Created attachment 93713 [details] [review] Bug 23007: Add TransfersBlockCirc system preference Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 93714 [details] [review] Bug 23007: Retain focus on 'barcode' input if transfers don't block circ To test: 1 - Apply patches 2 - Update database 3 - Confirm TransfersBlockCirc defaults to 'Block' 4 - Check in an item that needs transfer 5 - Confirm you must deal with the modal to proceed 6 - Repeat checking to get 'wrong-transfer-modal' 7 - Confirm you must deal with it to proceed 8 - Switch the pref to 'Don't block' 9 - Find a new item or cancel/fulfill the preceeding transfer 10 - Check in the item to generate transfer 11 - Note focus remains on the barcode field and you can continue scanning (or typing if you don't have a scanner) 12 - Note typing and pressing enter continues without requiring you to deal with modal 13 - Repeat for wrong transfer modal 14 - Verify hold and other modals still block continuing circulation Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Nice work! Pushed to master for 19.11.00