Web client transit list: odd default columns related to call numbers

Bug #1838326 reported by Jane Sandberg
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Status tracked in Master
3.3
Won't Fix
Undecided
Unassigned
3.4
Won't Fix
Undecided
Unassigned
3.5
Won't Fix
Undecided
Unassigned
Master
New
Undecided
Jeff Godin

Bug Description

The Web client transit list, by default, has Call Number Prefix and Call Number Suffix columns enabled, but Call Number Label column disabled.

The Call Number Label column is much more likely to be useful than the prefix and suffix ones. Could we switch it so that the affix columns are disabled by default, and the label column is enabled by default?

Our circ coordinator also gave the feedback that the term "Call Number Label" is confusing; she'd much prefer it just said "Call Number".

tags: added: bitesize
Revision history for this message
Jane Sandberg (sandbergja) wrote :

For anybody who wants to tackle this, you will probably want to work on lines 55-57 of this file: https://github.com/evergreen-library-system/Evergreen/blob/master/Open-ILS/src/templates/staff/circ/transits/t_list.tt2

Revision history for this message
Timothy Means (tmeans-ncc) wrote :
tags: added: pullrequest
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Looks good to me, Tim. Thanks for the branch!

I've signed off on your commit, and added a small follow-up commit in my branch user/sandbergja/lp1838326_web_client_transit_list_callnumbers

Here's a link: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sandbergja/lp1838326_web_client_transit_list_callnumbers

tags: added: signedoff
Michele Morgan (mmorgan)
Changed in evergreen:
milestone: none → 3.5-alpha
Michele Morgan (mmorgan)
Changed in evergreen:
milestone: 3.5-beta → 3.5.0
Changed in evergreen:
milestone: 3.5.0 → 3.5.1
Changed in evergreen:
milestone: 3.5.1 → 3.5.2
Changed in evergreen:
milestone: 3.5.2 → 3.6.1
Revision history for this message
Michele Morgan (mmorgan) wrote :

This patch has a conflict with current master.

The approach here shows the call number label by default, which was sorely needed, and it hides the prefix and suffix columns. Certainly an improvement, but it would be great if the call number column could be handled as it is in Open-ILS/src/templates/staff/circ/holds/t_pull_list.tt2, where the prefix/label/suffix all display in the call number column. For libraries that use prefixes and suffixes, this will display their entire call number in the one column.

adding a needsrebase tag

tags: added: needsrebase
removed: pullrequest signedoff
no longer affects: evergreen/3.6
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers