TPAC - show total copy count in record summary

Bug #1050039 reported by Ben Shum
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Confirmed
Wishlist
Unassigned

Bug Description

Evergreen master

Similar to how the Results page shows the number of currently available out of number total (3 of 5 copies available at X), we should do the same in the record display for consistency. We had reports from libraries/patrons who complained that only seeing X available confused them, especially if that number read 0 (because everything was checked out, etc.)

Branch at: user/bshum/tpac_record_copy_counts

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/tpac_record_copy_counts

Tags: tpac
Ben Shum (bshum)
Changed in evergreen:
milestone: none → 2.3.0
Revision history for this message
Thomas Berezansky (tsbere) wrote :

I have signed off on Ben's commit, but added two of my own.

One changes the "Show" link to instead just be the entire line of text to hopefully make it more obvious.

The other draws the link even if there are no "Available" copies, because that isn't the only reason you might care to change depth.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/tpac_record_copy_counts

Revision history for this message
Ben Shum (bshum) wrote :

Removing pullrequest till I can have a chance to fully test tsbere's additional commits.

The second commit to check for non-available copies looks good. But the first additional commit to change the styling of the line to be a link on the entire text instead of the "Show" link on the side is producing some minor controversy here. I have to get more feedback/opinions from others on the new proposed styling.

tags: removed: pullrequest
Changed in evergreen:
milestone: 2.3.0 → 2.3.1
Changed in evergreen:
milestone: 2.3.1 → 2.4.0-alpha
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Wishlist
Revision history for this message
Ben Shum (bshum) wrote :

Making this just wishlist for master dev now.

no longer affects: evergreen/2.2
no longer affects: evergreen/2.3
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-alpha1 → 2.4.0-beta
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-beta → 2.4.0-rc
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-rc → 2.5.0-alpha
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.0-m1 → none
Revision history for this message
Ben Shum (bshum) wrote :

I consider the new solution by Kathy in bug 1481844 better, so using that instead.

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.