Web Client: egBasicComboBox problems filtering choices when typing (i.e. autosuggest)
Bug #1819233 reported by
Remington Steed
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)
tags: | added: usability webstaffclient |
Changed in evergreen: | |
importance: | Undecided → Low |
Changed in evergreen: | |
assignee: | nobody → Mike Risher (mrisher) |
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 |
tags: | added: angularjs |
Changed in evergreen: | |
milestone: | 3.7.2 → 3.7.3 |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
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