web client: Cannot set default search box in Z39.50

[Elaine Hardy]

In the XUL client, users can set a default search box in the Z39.50 interface For example, if the user primarily searches by ISBN first, that box can be the default so that each time the interface is open, their cursor is in that box. This functionality is not in the web client.

[Jane Sandberg (sandbergja)]

I can confirm this on 3.1.0. I'm not sure the old way was the absolute best way to do this from a usability standpoint, because it relied on catalogers to read the "tip" text and figure out what a "search field label" is. It would be nice to be able to scan in an ISBN without any clicking or tabbing when I get to the Z39.50 screen, though.

Elaine, do you think most catalogers would want the ISBN field to be the default? Or would there be a lot of variety in what defaults folks would want? I'm wondering if this would be as a more global setting that individual catalogers don't have to think about, or if it's really important that individual catalogers set this up for themselves.

[Elaine Hardy (ehardy)]

I think it is important for catalogers to be able to set themselves. Most copy catalogers may prefer ISBN as default and never change it; but, others might prefer title. It also depends on task. When I am doing database cleanup, I typically use the tcn. And it isn't unusual for me to change the default if I either switch tasks or have records with no OCLC based tcn to overlay.

[Sarah Childs (sarahc)]

It would probably make sense to default to ISBN out of the box, but still to allow catalogers to change the default as needed

[Jason Etheridge (phasefx)]

I have a fix for this here:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/phasefx/lp1739286

[Elaine Hardy (ehardy)]

Tested fix on PINES test server. I was able to set the default and return to the interface and have the default stick. I changed the default, did tasks with the interface, reloaded the page, closed the page and reopened and the default followed as expected. I also searched with multiple queries (title and author) and with different filters and everything worked as expected

[Chris Sharp (chrissharp123)]

Pushed to master, rel_3_0, and rel_3_1.

Thanks, Jason and Elaine!

[Chris Sharp (chrissharp123)]

Reverted this fix in master, rel_3_0 and rel_3_1 because of problems reported in bug 1782401. Sorry for the confusion!

[Jason Etheridge (phasefx)]

New branch up at collab/phasefx/lp1739286.rebased

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/phasefx/lp1739286.rebased

Thanks!

[Chris Sharp (chrissharp123)]

Works for me. Signoff branch at http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/csharp/lp1739286.rebased

[Mike Rylander (mrylander)]

Hrm, do we need to find and set all keys in scope.infocus to false before we set the current one to true?

[Mike Rylander (mrylander)]

Picked to master, rel_3_1 and rel_3_0, with followup commit to make sure only one radio option tries to gain focus.