From 91aee0743ce607987513b2823e23dfdbc6da2135 Mon Sep 17 00:00:00 2001
From: Yaroslav Afenkin <91559310+yaroslavafenkin@users.noreply.github.com>
Date: Mon, 4 Nov 2024 15:02:59 +0200
Subject: [PATCH] [JENKINS-74025] Extract inline JavaScript from
 `checkboxContent.jelly`

---
 .../unochoice/common/checkbox-content.js      | 19 ++++++++++++
 .../unochoice/common/checkboxContent.jelly    | 30 +++----------------
 2 files changed, 23 insertions(+), 26 deletions(-)
 create mode 100644 src/main/resources/org/biouno/unochoice/common/checkbox-content.js

diff --git a/src/main/resources/org/biouno/unochoice/common/checkbox-content.js b/src/main/resources/org/biouno/unochoice/common/checkbox-content.js
new file mode 100644
index 00000000..39d8add4
--- /dev/null
+++ b/src/main/resources/org/biouno/unochoice/common/checkbox-content.js
@@ -0,0 +1,19 @@
+window.addEventListener("DOMContentLoaded", () => {
+    document.querySelectorAll(".checkbox-content-data-holder").forEach((dataHolder) => {
+        const { maxCount, randomName } = dataHolder.dataset;
+
+        var height = 0;
+        var refElement = document.getElementById(`ecp_${randomName}_0`);
+        if (maxCount > 0 && refElement && refElement.offsetHeight != 0) {
+            for (var i = 0; i < maxCount; i++) {
+                height += refElement.offsetHeight + 3;
+            }
+        }
+        else {
+            height = maxCount * 25.5;
+        }
+
+        height = Math.floor(height);
+        document.getElementById(`ecp_${randomName}`).style.height = height + "px";
+    });
+});
diff --git a/src/main/resources/org/biouno/unochoice/common/checkboxContent.jelly b/src/main/resources/org/biouno/unochoice/common/checkboxContent.jelly
index 628ab211..f2109d0b 100644
--- a/src/main/resources/org/biouno/unochoice/common/checkboxContent.jelly
+++ b/src/main/resources/org/biouno/unochoice/common/checkboxContent.jelly
@@ -1,7 +1,8 @@
 <?jelly escape-by-default='true' ?>
 <j:jelly xmlns:j="jelly:core"
          xmlns:d="jelly:define"
-         xmlns:transitionWrapper="transitionWrapper">
+         xmlns:transitionWrapper="transitionWrapper"
+         xmlns:st="jelly:stapler">
     <d:taglib uri="transitionWrapper">
       <d:tag name="randomTableIdBlock">
           <j:choose>
@@ -101,29 +102,6 @@
       </j:forEach>
     </transitionWrapper:randomTableIdBlock>
   </div>
-<script type="text/javascript">
-<![CDATA[
-  (function() {
-      var f = function() {
-          var height = 0;
-          var maxCount = ${it.getVisibleItemCount()};
-
-          var refElement = document.getElementById("ecp_${h.escape(it.randomName)}_0");
-          if(maxCount > 0 && refElement && refElement.offsetHeight !=0) {
-              for(var i=0; i< maxCount; i++) {
-                  height += refElement.offsetHeight + 3;
-              }
-          }
-          else {
-              height = maxCount * 25.5;
-          }
-
-          height = Math.floor(height);
-          document.getElementById("ecp_${h.escape(it.randomName)}").style.height = height + "px";
-      };
-
-      f();
-  })();
-]]>
-  </script>
+  <span class="checkbox-content-data-holder" data-max-count="${it.getVisibleItemCount()}" data-random-name="${h.escape(it.randomName)}" />
+  <st:adjunct includes="org.biouno.unochoice.common.checkbox-content" />
 </j:jelly>