Web Client: egBasicComboBox problems filtering choices when typing (i.e. autosuggest)

Bug #1819233 reported by Remington Steed
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned
3.7
Fix Released
Low
Unassigned

Bug Description

Our AngularJS directive "egBasicComboBox" has some buggy behavior when it attempts to filter the options list based on what you type into the textbox.

This editable dropdown is used in multiple places in the web client. For example, on the Item Editor, this combobox is used for the Template selector. Also, bug 1793196 will use this combobox in the Volume Editor for the batch Call Number dropdown.

Examples of buggy behavior:

- Typing certain characters causes console errors, such as square bracket "["
- In the batch Call Number dropdown (as added in bug 1793196), typing a matching string (which should show choices) only displays an empty list of choices (a tiny white popup)

Remington Steed (rjs7)
tags: added: usability webstaffclient
Changed in evergreen:
importance: Undecided → Low
Mike Risher (mrisher)
Changed in evergreen:
assignee: nobody → Mike Risher (mrisher)
Revision history for this message
Mike Risher (mrisher) wrote :

I think I found the filtering bug. If you have an open list of values, then click on the caret ^ button to close the list it all works fine. However, if you close the list by clicking on the text input box, the list closes but filtering stops working properly. I fixed this bug. Also, it no longer shows console errors when values like [ are typed. It's no longer case sensitive. Typing "t" will match with "Tuesday", for example. Branch here:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrisher/lp1819233-eg-combo-box-filtering

tags: added: pullrequest
Changed in evergreen:
assignee: Mike Risher (mrisher) → nobody
Revision history for this message
Terran McCanna (tmccanna) wrote :
tags: added: signedoff
Changed in evergreen:
milestone: none → 3.6.1
Changed in evergreen:
milestone: 3.6.1 → 3.6.2
Changed in evergreen:
milestone: 3.6.2 → 3.6.3
tags: added: eg-grid
removed: webstaffclient
Changed in evergreen:
milestone: 3.6.3 → 3.6.4
Changed in evergreen:
milestone: 3.6.4 → 3.7.2
Gina Monti (gmonti90)
tags: added: angularjs
Changed in evergreen:
milestone: 3.7.2 → 3.7.3
Revision history for this message
Galen Charlton (gmc) wrote :

Tested and pushed down to rel_3_7. Thanks, Mike and Terran!

no longer affects: evergreen/3.6
Changed in evergreen:
milestone: 3.7.3 → 3.8.1
status: New → Confirmed
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.