Rows out of order in OPAC circ history

Bug #2015481 reported by Jason Boyer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.10
Fix Released
Medium
Unassigned
3.9
Fix Released
Medium
Unassigned

Bug Description

Eg 3.9+
The order of the columns in the BPAC circ history doesn't match the order on the Items Out page, while the CSS for history on narrow screens *does* match the items out page, so on narrow screens the labels and data don't line up. This is because Call Number comes after Author in the horizontal history view but at the bottom of the narrow view. To keep things consistent the CN should be moved to the end of the circ history row to match items out and the narrow screen labels.

Revision history for this message
Jason Boyer (jboyer) wrote :

Fix here: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jboyer/lp2015481_skinny_labels / working/user/jboyer/lp2015481_skinny_labels

Testing is easy:
Login to the opac as a user and turn on circ history
Login to staff client and check an item out to the user
Login to the opac again as the user and view circ history
Make the browser window narrow enough to engage the vertical item display
Note that the labels and data don't make sense after "Author"
Apply patch
Look at user history in a narrow view again, note that things make much more sense now

tags: added: pullrequest
Jason Boyer (jboyer)
Changed in evergreen:
milestone: none → 3.next
importance: Undecided → Medium
Jason Boyer (jboyer)
no longer affects: evergreen/3.11
Changed in evergreen:
milestone: 3.next → none
tags: added: opac-account
Revision history for this message
Galen Charlton (gmc) wrote :

Signed off: user/gmcharlt/lp2015481_signoff

tags: added: sig
Changed in evergreen:
milestone: none → 3.11-beta
status: New → Confirmed
tags: added: signedoff
removed: sig
Michele Morgan (mmorgan)
Changed in evergreen:
assignee: nobody → Michele Morgan (mmorgan)
Revision history for this message
Michele Morgan (mmorgan) wrote :

Looks great! Pushed to main, rel_3_11, rel_3_10, and rel_3_9.

Thanks Jason and Galen!

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Michele Morgan (mmorgan) → nobody
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.