From 4d8b078e99a081d3a9f205e074f7cb3327e57ab2 Mon Sep 17 00:00:00 2001
From: Doom <doomsniper09@gmail.com>
Date: Sun, 28 Dec 2025 14:21:53 +0200
Subject: [PATCH] Prevent duplicate script injection in tabbed_block plugin

The tabbed_block plugin was injecting the same JavaScript code every time a tabbed block was rendered. On pages with multiple tabbed blocks, this resulted in duplicate script tags and function definitions.

This change uses a context flag to ensure the script is only included once per page, reducing page size and avoiding redundant code.
---
 plugins/tabbed_block.rb | 50 ++++++++++++++++++++++++-----------------
 1 file changed, 29 insertions(+), 21 deletions(-)

diff --git a/plugins/tabbed_block.rb b/plugins/tabbed_block.rb
index 508dbf54940f..59abdab18865 100644
--- a/plugins/tabbed_block.rb
+++ b/plugins/tabbed_block.rb
@@ -31,28 +31,36 @@ def render(context)
       site = context.registers[:site]
       converter = site.find_converter_instance(::Jekyll::Converters::Markdown)
 
-      <<~MARKUP
-        <script>
-        function openTab(tab){
-          const tabKey = tab.querySelector("div").id;
-          const targetTabContent = tab.parentElement.parentElement.querySelector(`#${tabKey}.tabbed-content-block-content`);
-          const tabContents = tab.parentElement.parentElement.querySelectorAll(".tabbed-content-block-content")
-
-          tabContents.forEach((content) => {
-            content.style.display = "none"
-          });
-          targetTabContent.style.display = "block";
-        }
-        window.addEventListener('DOMContentLoaded', (event) => {
-          const tabbedBlocks = document.querySelectorAll(".tabbed-content-block");
-          tabbedBlocks.forEach((block) => {
-              block.querySelector("input").checked = true;
-              block.querySelector(".tabbed-content-block-content").style.display = "block";
+      # Only include the script once per page to avoid duplicate code
+      script = ""
+      unless context['tabbed_block_script_included']
+        context['tabbed_block_script_included'] = true
+        script = <<~SCRIPT
+          <script>
+          function openTab(tab){
+            const tabKey = tab.querySelector("div").id;
+            const targetTabContent = tab.parentElement.parentElement.querySelector(`#${tabKey}.tabbed-content-block-content`);
+            const tabContents = tab.parentElement.parentElement.querySelectorAll(".tabbed-content-block-content")
+
+            tabContents.forEach((content) => {
+              content.style.display = "none"
+            });
+            targetTabContent.style.display = "block";
+          }
+          window.addEventListener('DOMContentLoaded', (event) => {
+            const tabbedBlocks = document.querySelectorAll(".tabbed-content-block");
+            tabbedBlocks.forEach((block) => {
+                block.querySelector("input").checked = true;
+                block.querySelector(".tabbed-content-block-content").style.display = "block";
+            });
+
           });
-      
-        });
-        </script>
-        <div class="tabbed-content-block">
+          </script>
+        SCRIPT
+      end
+
+      <<~MARKUP
+        #{script}<div class="tabbed-content-block">
           #{render_tabbed_block(vars: vars, converter: converter)}
         </div>
       MARKUP