web client: type to select wonky in item stat cat dropdowns

Bug #1751356 reported by Andrea Neiman
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Won't Fix
Medium
Unassigned

Bug Description

(Related bug 1751353 is for patron stat cats)

XUL client has type-to-select for stat cats in both the item editor & patron editor.

In the web client item editor, this still exists but works a little wonkily. It doesn't show the characters as you type, and it only seems to work if a stat cat value is unset or reset back to no value.

Revision history for this message
Erica Rohlfs (erohlfs) wrote :

I tested on 3.0.3. When testing, I cannot recreate this behavior for patron stat cats, even with the caveat of unset value and/or reset back to no value. What I'm observing is the inability to type-to-select without any additional selection being made. I tested both registering a patron and an existing patron account. Within the item editor, however, I am seeing the behavior you mention on item stat cats. What I see is that, instead of the filtering options populating, the select option jumps down to the beginning of the letter typed. For example, if I type "k" in the item stat cat dropdown, it will jump down to the start of k's in my list, such as kit. I see similar behavior in other interfaces not related to stat cats, such as typing "k" to select through all of the different libraries starting with k when selecting a pickup library location. My best description of the behavior is like that of a "Find on this Page" feature and less like a filter.

Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
Galen Charlton (gmc)
tags: added: webstaffclient
Revision history for this message
Remington Steed (rjs7) wrote :

For anyone who wants to work on this, it sounds like the Stat Cat dropdown needs to be replaced with an eg-basic-combo-box. See bug #1793196 for one example of how this can be done.

Mike Risher (mrisher)
Changed in evergreen:
assignee: nobody → Mike Risher (mrisher)
Revision history for this message
Mike Risher (mrisher) wrote :

The item stat cat dropdown is now using an eg-basic-combo-box. Here is the branch:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrisher/lp1751356-item-stat-cat-dropdowns

Please note that eg-basic-combo-box has issues with filtering, which have been reported here:
https://bugs.launchpad.net/evergreen/+bug/1819233

tags: added: pullrequest
Changed in evergreen:
assignee: Mike Risher (mrisher) → nobody
Revision history for this message
Rogan Hamby (rogan-hamby) wrote :

sign off pushed to user/rogan/lp1751356_signoff

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

This needs a bit more work, as it breaks the statcat dropdown when using the volcopy editor to edit an item template rather than an item.

tags: added: needsrepatch
removed: pullrequest signedoff
Revision history for this message
Mike Risher (mrisher) wrote :

Thanks for the feedback Galen. The dropdowns are now working properly on the item template. Also, during testing I came across a scenario where the end user switches from a saved value for a stat cat to no saved value. To indicate no selection I added the entry "<NONE>". This is consistent with with the text on other fields that are unset.

Branch here:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrisher/lp1751356-item-stat-cat-dropdowns

tags: added: pullrequest
Mike Risher (mrisher)
tags: removed: needsrepatch
Revision history for this message
Jennifer Bruch (jbruchpails) wrote :

Any chance this fix can be applied to the Patron Stat Cats as well?

(Related bug 1751353 is for patron stat cats)

Revision history for this message
Christine Burns (christine-burns) wrote :

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

https://terran-master.gapines.org/eg/staff/

tags: added: signedoff
Michele Morgan (mmorgan)
Changed in evergreen:
milestone: none → 3.6.2
Changed in evergreen:
milestone: 3.6.2 → 3.6.3
tags: added: statcats
removed: webstaffclient
Changed in evergreen:
milestone: 3.6.3 → 3.6.4
Changed in evergreen:
milestone: 3.6.4 → 3.7.2
Changed in evergreen:
milestone: 3.7.2 → 3.7.3
Changed in evergreen:
milestone: 3.7.3 → none
no longer affects: evergreen/3.6
Revision history for this message
Jane Sandberg (sandbergja) wrote :

With appreciation to Mike and Christine, I am marking this as Won't Fix, since it doesn't appear to be an issue in the Angular holdings editor.

Changed in evergreen:
status: Confirmed → Won't Fix
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.