From b0a16847fedcf8edc010247e8898ddfae90a1b3a Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Mon, 6 Feb 2023 13:10:30 +0000 Subject: [PATCH] Bug 31413: Set focus for cursor to Selector when adding a new audio alert This patch adds a step to the process of displaying the new audio alert form so that focus will be set to the first field when the form has finished appearing. The typical "focus" class only works on elements which are present in the initial DOM when the page is displayed. To test, apply the patch and go to Administration -> Audio alerts. Click the "New alert" button. When the form appears cursor focus should be on the "Selector" field. Also test the edit process: When editing an existing alert there should not be automatic cursor focus. Signed-off-by: Lucas Gass Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 7c8434553a29b177b879f958c59487cf854c3c13) Signed-off-by: Matt Blenkinsop (cherry picked from commit 5e593d098dab2f33a7a7ade03b7af64801765d8b) Signed-off-by: Lucas Gass --- koha-tmpl/intranet-tmpl/prog/js/audio_alerts.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/js/audio_alerts.js b/koha-tmpl/intranet-tmpl/prog/js/audio_alerts.js index 0fa22dc430..7c27fd45e6 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/audio_alerts.js +++ b/koha-tmpl/intranet-tmpl/prog/js/audio_alerts.js @@ -25,7 +25,9 @@ $( document ).ready(function() { $("#newalert").on("click",function(e){ e.preventDefault(); - $("#new-alert-form").show(); + $("#new-alert-form").show( 0, function(){ + $("#selector").focus(); + }); $("#toolbar, #delete-alert-form").hide(); }); -- 2.20.1