From 82d3dcb084b785d2852171d57af72666abe433b9 Mon Sep 17 00:00:00 2001
From: Pontoporeia <pontoporeia@happyngreen.fr>
Date: Wed, 24 Jun 2026 13:18:14 +0200
Subject: [PATCH] Fix repertoire column scrolling + admin TOC duplication
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- repertoire.css: add min-height: 0 to column <ul> scroll containers so
  grid 1fr row shrinks below content and overflow-y: auto activates
- admin-toc.js: add __adminTocBuilt guard + nav.children check to prevent
  double population when loaded both via admin.min.js and direct <script>
- admin-toc.php: remove duplicate <script src="admin-toc.js"> tag —
  JS is already bundled in admin.min.js
---
 app/public/assets/css/repertoire.css       | 1 +
 app/public/assets/js/app/admin-toc.js      | 8 ++++++++
 app/templates/admin/partials/admin-toc.php | 4 +---
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/app/public/assets/css/repertoire.css b/app/public/assets/css/repertoire.css
index 82adbf1..c71eb7c 100644
--- a/app/public/assets/css/repertoire.css
+++ b/app/public/assets/css/repertoire.css
@@ -73,6 +73,7 @@
     grid-row: 2;
     overflow-y: auto;
     overflow-x: hidden;
+    min-height: 0;
     padding-top: var(--space-2xs);
     padding-bottom: var(--space-l);
     padding-left: 0;
diff --git a/app/public/assets/js/app/admin-toc.js b/app/public/assets/js/app/admin-toc.js
index b50ab2f..59658e2 100644
--- a/app/public/assets/js/app/admin-toc.js
+++ b/app/public/assets/js/app/admin-toc.js
@@ -4,14 +4,22 @@
  *
  * Renders nav links from section[aria-labelledby] headings inside #main-content.
  * Hides the TOC aside if fewer than 2 sections exist.
+ *
+ * Guarded: only runs once even if loaded multiple times (e.g. via bundle + direct <script>).
  */
 (function () {
+    if (window.__adminTocBuilt) return;
+    window.__adminTocBuilt = true;
+
     function build() {
         var main = document.getElementById('main-content');
         var nav = document.getElementById('admin-toc-list');
         var aside = document.getElementById('admin-toc');
         if (!main || !nav || !aside) return;
 
+        // Guard against double population (nav already has children from a prior run)
+        if (nav.children.length > 0) return;
+
         var sections = main.querySelectorAll('section[aria-labelledby]');
         if (sections.length < 2) {
             aside.hidden = true;
diff --git a/app/templates/admin/partials/admin-toc.php b/app/templates/admin/partials/admin-toc.php
index c57d9d7..d6bf448 100644
--- a/app/templates/admin/partials/admin-toc.php
+++ b/app/templates/admin/partials/admin-toc.php
@@ -8,8 +8,6 @@
 ?>
 <aside id="admin-toc" class="admin-toc" aria-label="Sur cette page">
     <nav class="admin-toc-list" id="admin-toc-list">
-        <!-- populated by JS -->
+        <!-- populated by JS (admin-toc.js, loaded via admin.min.js) -->
     </nav>
 </aside>
-
-<script src="<?= App::assetV('/assets/js/app/admin-toc.js') ?>"></script>