web client: progress indicator needed in patron search

Bug #1565009 reported by Kathy Lussier
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned
2.10
Fix Released
Undecided
Unassigned
2.11
Fix Released
Undecided
Unassigned

Bug Description

The patron search could use a progress indicator letting staff know the search is still in progress.

If staff is working from an existing list patrons and conducts a new search, the first thing the interface does is clear the existing list and replace it with a "No Items to Display" message. If the search takes a long time, it's hard to tell if the search yielded zero results or if the search is still working.

Alternatively, we could choose not display the "No Items to Display" message until the search is complete, but I think progress indicators are a good UI element to add for any search or for screens that take a long time to load.

Revision history for this message
Bill Erickson (berick) wrote :

Patron search progress bar pushed:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1565009-webstaff-patron-search-progress

From the commit:

Displays a progress bar while waiting for results from a patron search in the browser client. Bar only displays when a server-side search is required. (It's not displayed for cached searches, etc.). Results grid is hidden while the progress bar is visible and vice versa.

==

Note that no attempt is made to indicate actual progress, since the grid begins rendering with the first user retrieved, only that a search is in progress.

Changed in evergreen:
status: New → Confirmed
milestone: none → 2.11-rc
tags: added: pullrequest
Revision history for this message
Jeanette Lundgren (jlundgren) wrote :

I plan to test this on Bug Squashing Day

Changed in evergreen:
assignee: nobody → Jeanette Lundgren (jlundgren)
Revision history for this message
Jeanette Lundgren (jlundgren) wrote :

This looks great! Thanks Bill and Kathy.

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

Changed in evergreen:
assignee: Jeanette Lundgren (jlundgren) → nobody
tags: added: signedoff
Changed in evergreen:
milestone: 2.11-rc → 2.11.1
Revision history for this message
Kathy Lussier (klussier) wrote :

Thank you Bill and Jeanette! Signed off and merged to master, 2.11 and 2.10.

Changed in evergreen:
milestone: 2.11.1 → 2.next
status: Confirmed → Fix Committed
Changed in evergreen:
milestone: 2.next → 2.12-beta
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.