From f3742afa6444ed8f3b2084d73b5aab154677a046 Mon Sep 17 00:00:00 2001
From: Lucas Gass <lucas@bywatersolutions.com>
Date: Wed, 18 Oct 2023 20:31:32 +0000
Subject: [PATCH] Bug 35101: Stop screen from jumping back to top when using
 barcode.pl plugin

To test:
1. Set 'autoBarcode' to something other than 'don't generate automatically'
2. If the a framwwork make sure some 952 fields have the barcode.pl plugin enabled
3. Add a new item and click the '...' next to the 952 field with the barcode.pl plugin enabled.
4. The screen jumps back to the top, annoying.
5. Apply patch
6. Try again, it should not jump anymore!

Signed-off-by: Barbara Johnson <barbara.johnson@swbell.net>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit eec5d7e4be7d092f07e47468c50a4ec48b842dc1)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit 85d38e5ef587657ba5681825cbd0c5fda5094047)
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
---
 cataloguing/value_builder/barcode.pl | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/cataloguing/value_builder/barcode.pl b/cataloguing/value_builder/barcode.pl
index bd4a927ef1..b166b46c4d 100755
--- a/cataloguing/value_builder/barcode.pl
+++ b/cataloguing/value_builder/barcode.pl
@@ -92,10 +92,12 @@ $scr
 
 function Focus$function_name(event) {
     set_barcode(event.data.id, false);
+    return false;
 }
 
 function Click$function_name(event) {
     set_barcode(event.data.id, true);
+    return false;
 }
 </script>
 END_OF_JS
-- 
2.39.5