From c2345e04fa58aea273b9331f72a6b8d54e10f8ac Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Wed, 26 Feb 2020 15:53:48 +0000 Subject: [PATCH] Bug 24510: When Placing a Hold, cursor doesn't focus on Patron Name This alternate patch tries to implement a fix in a more "correct" way by tying the cursor focus to the initialization of the jQueryUI tabs. By configuring both the "create" and "activate" events we can make sure the correct input field is focused. I think the only advantage of this patch over the other one is that tying the focus to the tabs activation makes it a little more obvious where in the DOM loading sequence the focus is being set. To test, follow the original patch's test plan: TEST PLAN: 1. Go to Biblio Record 2. Place Hold (request.pl) - either the Hold above the Bib or the Hold on the left, it does the same thing. 3. Note that the cursor does not go to Patron Name (for whom to place the hold for), it goes to the Top Search bar under Checkout. 4. Apply patch and reload the page. 5. The focus should now be correctly set. 6. Toggle between 'Patrons' and 'Club' tabs. 7. Focus should stay set. Signed-off-by: Jonathan Druart --- .../prog/en/modules/reserve/request.tt | 31 +++++++++++++--------- 1 file changed, 18 insertions(+), 13 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt index b393d7aad5..6bb34c5476 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -978,9 +978,25 @@ $(document).ready(function() { [% UNLESS clubs %] - $('#circ_holds_select').tabs({active: 0}); + $('#circ_holds_select').tabs({ + active: 0, + activate: function(){ + $(this).find("input.focus").focus(); + }, + create: function(){ + $(this).find("input.focus").focus(); + } + }); [% ELSE %] - $('#circ_holds_select').tabs({active: 1}); + $('#circ_holds_select').tabs({ + active: 1, + activate: function(){ + $(this).find("input.focus").focus(); + }, + create: function(){ + $(this).find("input.focus").focus(); + } + }); [% END %] function ToggleHoldsToPlace() { if ( $("#requestany").prop('checked') ) { @@ -1261,17 +1277,6 @@ [% END %] [% END %] - if ( $(".focus").is(":visible") ) { - $(".focus").focus(); - } - - $('[href="#holds_patronsearch_pane"]').click( function() { - $('#patron').focus(); - }) - - $('[href="#holds_clubsearch_pane"]').click( function() { - $('#club').focus(); - }) }); [% END %] -- 2.11.0