Apply button in Patron Bill History screen is in a confusing location

[Jane Sandberg]

The apply button in the Patron Bill History screen is located before the two datepickers (screenshot attached). In most forms, the apply/submit button is placed *after* form fields. This has led our staff to completely miss that there is an Apply button at all.

A better UX would be to place the apply button after the datepickers, in the flow that users will be more accustomed to. A quick glance suggests that this could be accomplished by rearranging some lines in this file: https://github.com/evergreen-library-system/Evergreen/blob/master/Open-ILS/src/templates/staff/circ/patron/t_bill_history.tt2

An even better UX would be to remove the apply button completely, since it is an extra step -- and refresh the grid every time the user selects a new date.

[Terran McCanna (tmccanna)]

+1 to refreshing the grid automatically.

[Dan Briem (dbriem)]

I tested this on 3.3.2. I can see how someone might not see the apply button and I noticed invalid date ranges are possible.

This is a potential patch for testing that eliminates the apply button and watches the dates. If they are valid date objects in a valid date range the new date range is applied and the grid refreshes. If the start date > finish date they are set equal to each other to force a valid date range.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbriem/lp1841089_remove_bill_history_apply_btn

[Jane Sandberg (sandbergja)]

This works well for me. Thanks, Dan! Sign off branch here: user/sandbergja/lp1841089_remove_bill_history_apply_btn_signoff

[Galen Charlton (gmc)]

Pushed all the way to rel_3_1. Thanks, Dan and Jane!