Wishlist: Move strict barcode checkbox

Bug #1990968 reported by Elizabeth Thomsen
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

I think it would be easier for circulation staff to keep track of whether or not the Strict Barcode option is checked if it were relocated to the be directly under or next to the barcode box on both the Checkout and Checkin screens. In both cases, it's located below the grid of items in busy areas of the screen with other options that (mostly) refer to receipts and completing the transaction.

Revision history for this message
Elizabeth Thomsen (et-8) wrote :
tags: added: bitesize
Changed in evergreen:
assignee: nobody → Scott Angel (scottangel1)
Revision history for this message
Scott Angel (scottangel1) wrote :

Fairly straightforward fix here. Just moved some html around. This is my first commit to Evergreen.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/scottangel/LP1990968_move_strict_barcode_checkbox

tags: added: pullrequest
Changed in evergreen:
assignee: Scott Angel (scottangel1) → nobody
Changed in evergreen:
importance: Undecided → Wishlist
status: New → Confirmed
milestone: none → 3.11-beta
Changed in evergreen:
assignee: nobody → Jason Stephenson (jstephenson)
Revision history for this message
Jason Stephenson (jstephenson) wrote :

I have pushed a branch to https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/dyrcona/LP1990968_move_strict_barcode_checkbox_signoff that includes my sign-off on Scott's commit and an additional commit to move the strict barcode checkbox for the Angular staff client interfaces.

working/collab/dyrcona/LP1990968_move_strict_barcode_checkbox_signoff

Changed in evergreen:
assignee: Jason Stephenson (jstephenson) → nobody
tags: added: usability
Revision history for this message
Jason Stephenson (jstephenson) wrote :

I pushed a collab branch to the working repository that can make testing the Angular/eg2 changes easier. The branch presently has one commit that modifies the Circulation menu entries for Checkout, Checking, and Renew Items to go to the Angular client instead of AngularJS.

working/collab/dyrcona/enable_Angular

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/dyrcona/enable_Angular

If you use this branch for testing, be sure NOT to push the change to master or other branches.

Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

Tested on bug squashing server and I see no change to where the Strict Barcode box is located in angularjs or angular.

Revision history for this message
Susan Morrison (smorrison425) wrote :

Testing on https://bugsquash.mobiusconsortium.org/eg2/en-US/staff and I don't see any difference in the location of Strict Barcode in either the Check In or Patron > Check Out interfaces.

Revision history for this message
Blake GH (bmagic) wrote :

Susan,

That was my fault. I didn't actually apply the patch to the server. I applied the patch from comment 4 and didn't realize that this wasn't the patch for this bug. (I didn't read very carefully). I've now applied the patch properly now! Please retest.

Revision history for this message
Jennifer Pringle (jpringle-u) wrote (last edit ):

I now see the strict barcode checkbox beside the barcode field in the angular Check In and Check Out; however, the strict barcode checkbox hasn't moved in the angularjs Check In and Check Out.

Looking at the new position in the angular interfaces I'm undecided as to whether this will make things easier or whether staff will be more likely to check the box by accident since it's more prominent.

I've created a new bug report for a wishlist to have an option to hide the checkbox altogether on a per library basis. (I think we've only ever had libraries use it by accident.) https://bugs.launchpad.net/evergreen/+bug/2008834

Revision history for this message
Jason Stephenson (jstephenson) wrote :

Jennifer, did you clear cache and hard reload when testing AngularJS the second time? I ask because the AngularJS change worked for me before I signed off on it. Something may have changed in the meantime...

Revision history for this message
Brett French (bsfrench) wrote :

Tested on https://bugsquash.mobiusconsortium.org/eg/staff and everything looks good as far as I can tell. Strict Barcode checkbox is appearing below the barcode field on the checkout, checkin, and renew items screens.

Revision history for this message
Elizabeth Thomsen (et-8) wrote :

I have tested this code and consent to signing off on it with my name, Elizabeth Thomsen, and my email address, <email address hidden>.

tags: added: signedoff
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed for inclusion in 3.11. Thanks, Scott, Jason, Brett, and Elizabeth!

Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.