From e867769d0971c2982ecb32dc74dc5fe2925bec38 Mon Sep 17 00:00:00 2001 From: Janusz Kaczmarek Date: Thu, 21 Mar 2024 16:20:24 +0100 Subject: [PATCH] Bug 36388: Mouse operation does not work in draggable fields in authority editor When using the authority editor in Firefox, it is not possible to set the cursor or select text from the text field. This patch mimics the corrections made in bug 35425 for biblio editor. Test plan: ========== 1. Start editing an existing authority record and try to put the cursor with the mouse in the middle of the content of any subfield or to select text with the mouse. It should not be possilbe. 2. Apply the patch (reload plack etc.). 3. Repeat p. 1 - everything should work as expected. Signed-off-by: Lucas Gass Signed-off-by: Pedro Amorim Signed-off-by: Victor Grousset/tuxayo Signed-off-by: Katrin Fischer (cherry picked from commit 6e09fb6ad8eaab2e219c01cc332b404ad89088c0) Signed-off-by: Fridolin Somers --- .../prog/en/modules/authorities/authorities.tt | 17 ++++++----------- .../prog/en/modules/cataloguing/addbiblio.tt | 9 --------- koha-tmpl/intranet-tmpl/prog/js/cataloging.js | 10 ++++++++++ 3 files changed, 16 insertions(+), 20 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt index 94be2063b5..d576d4e2db 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt @@ -48,17 +48,8 @@ selectTab( "#tab0XX_panel" ); } - $("#authoritytabs ul.sortable_field").each( (i, e) => { - Sortable.create(e, { - animation: 150 - }); - }); - - $("#authoritytabs ul.sortable_subfield").each( (i, e) => { - Sortable.create(e, { - animation: 150 - }); - }); + initializeSortable("#authoritytabs ul.sortable_field"); + initializeSortable("#authoritytabs ul.sortable_subfield"); Sticky = $("#toolbar"); Sticky.hcSticky({ @@ -556,6 +547,9 @@
[% UNLESS hide_marc %] + [% IF ( innerloo.repeatable ) %] + + [% END %] [% IF advancedMARCEditor %] [% innerloo.tag | html %] [% ELSE %] @@ -638,6 +632,7 @@ [% UNLESS hide_marc %]
+ { - Sortable.create(e, { - handle: '.handle', - direction: 'vertical', - animation: 150, - }); - }); - } initializeSortable("#addbibliotabs ul.sortable_field"); initializeSortable("#addbibliotabs ul.sortable_subfield"); diff --git a/koha-tmpl/intranet-tmpl/prog/js/cataloging.js b/koha-tmpl/intranet-tmpl/prog/js/cataloging.js index dc10f28c45..4eac537d6c 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/cataloging.js +++ b/koha-tmpl/intranet-tmpl/prog/js/cataloging.js @@ -603,6 +603,16 @@ function CheckImportantSubfields(p){ return total; } +function initializeSortable(selector) { + $(selector).each((i, e) => { + Sortable.create(e, { + handle: '.handle', + direction: 'vertical', + animation: 150, + }); + }); +} + $(document).ready(function() { $("input.input_marceditor, input.indicator").addClass('noEnterSubmit'); $(document).ajaxSuccess(function() { -- 2.39.5