From a445b72a6e5a2cf1479f7593cd8e5ca739b2f1b9 Mon Sep 17 00:00:00 2001
From: william <you@example.com>
Date: Sun, 30 Aug 2026 11:48:25 +0200
Subject: [PATCH] Corrige "Ouvrir"/"Modifier" inertes dans l'onglet Blocs de
 logique
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Deux bugs distincts, signalés par capture d'écran (bouton "Ouvrir" sans
effet, ligne d'édition affichée en permanence au lieu d'être cachée) :

1. onclick="openLogicBlockPanel({{ b.id }}, {{ b.name|tojson }})" cassait
   l'attribut HTML : tojson produit des guillemets DOUBLES (valides en
   JSON), qui terminaient prématurément l'attribut onclick="..." lui-même
   entre guillemets doubles — le gestionnaire de clic généré était donc
   tronqué et invalide, provoquant une erreur JS non interceptée qui
   arrêtait aussi tout le script restant dans la même balise <script>
   (dont l'IIFE qui devait poser window.openLogicBlockPanel). Corrigé en
   ne passant que l'id dans l'attribut et en retrouvant le nom du bloc
   côté client depuis FLOW_BLOCKS (déjà chargé) — plus aucune chaîne
   utilisateur à échapper dans un attribut HTML.

2. <tr class="hidden" id="blockEditRow..."> ne se cachait jamais : le
   CSS ne définissait .hidden que scopé (.floatPanel.hidden,
   .columnFilterMenu.hidden), jamais en règle générique — ajoutée dans
   styles/forge-custom.css.

Vérifié : 215 tests passent, syntaxe JS validée sur un scénario avec un
vrai bloc existant (reproduisant exactement la situation signalée),
onclick généré inspecté directement dans le HTML rendu.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
---
 static/style.css           |  5 +++++
 styles/forge-custom.css    |  5 +++++
 templates/screen_edit.html | 12 +++++-------
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/static/style.css b/static/style.css
index 60bf9f27..c267766e 100644
--- a/static/style.css
+++ b/static/style.css
@@ -404,6 +404,11 @@ body.builderBody > main.content{ flex:1 1 auto; min-height:0; overflow:hidden; d
   box-shadow:0 12px 40px rgba(0,0,0,.55);
 }
 .floatPanel.hidden{ display:none; }
+/* Utilitaire générique (jusqu'ici .hidden n'existait que scopé à
+   .floatPanel/.columnFilterMenu — ex. la ligne d'édition d'un bloc de
+   logique dans screen_edit.html, un <tr>, avait besoin d'un .hidden qui
+   marche seul, sans classe compagne). */
+.hidden{ display:none; }
 /* Coin de redimensionnement (voir wireFloatPanelResize() dans
    game_dashboard.html) — un panneau devient redimensionnable simplement en
    lui ajoutant ce coin, JS mis à part rien d'autre à faire côté CSS. */
diff --git a/styles/forge-custom.css b/styles/forge-custom.css
index dfb4729f..9f03893d 100644
--- a/styles/forge-custom.css
+++ b/styles/forge-custom.css
@@ -212,6 +212,11 @@ body.builderBody > main.content{ flex:1 1 auto; min-height:0; overflow:hidden; d
   box-shadow:0 12px 40px rgba(0,0,0,.55);
 }
 .floatPanel.hidden{ display:none; }
+/* Utilitaire générique (jusqu'ici .hidden n'existait que scopé à
+   .floatPanel/.columnFilterMenu — ex. la ligne d'édition d'un bloc de
+   logique dans screen_edit.html, un <tr>, avait besoin d'un .hidden qui
+   marche seul, sans classe compagne). */
+.hidden{ display:none; }
 /* Coin de redimensionnement (voir wireFloatPanelResize() dans
    game_dashboard.html) — un panneau devient redimensionnable simplement en
    lui ajoutant ce coin, JS mis à part rien d'autre à faire côté CSS. */
diff --git a/templates/screen_edit.html b/templates/screen_edit.html
index 8407a9e3..14e27d9f 100644
--- a/templates/screen_edit.html
+++ b/templates/screen_edit.html
@@ -216,7 +216,7 @@
               <td>{% if b.element_labels %}{{ b.element_labels|join(', ') }}{% else %}<span class="hint">—</span>{% endif %}</td>
               <td>{{ b.node_count }}</td>
               <td class="fieldsTableActions">
-                <button type="button" class="button is-small is-primary" onclick="openLogicBlockPanel({{ b.id }}, {{ b.name|tojson }})" title="Ouvrir le graphe de ce bloc">🧩 Ouvrir</button>
+                <button type="button" class="button is-small is-primary" onclick="openLogicBlockPanel({{ b.id }})" title="Ouvrir le graphe de ce bloc">🧩 Ouvrir</button>
                 <button type="button" class="button is-small" onclick="toggleBlockEditRow({{ b.id }})" title="Modifier le nom/la description">✏️</button>
                 <form method="post" action="{{ url_for('flow_block_delete', slug=game.slug, screen_id=screen.id, block_id=b.id) }}"
                       onsubmit="return confirm('Supprimer le bloc « {{ b.name }} » ? Cela supprime aussi ses {{ b.node_count }} nœud(s) et leurs connexions.');" style="display:inline;">
@@ -2963,10 +2963,11 @@ function makeFloatPanelFullscreenable(panel, toggleBtn) {
   makeFloatPanelDraggable(panel, document.getElementById('logicBlockPanelHeader'));
   makeFloatPanelResizable(panel, document.getElementById('logicBlockPanelResizeHandle'));
   makeFloatPanelFullscreenable(panel, panel.querySelector('.floatPanelFullscreenBtn'));
-  window.openLogicBlockPanel = function (blockId, blockName) {
+  window.openLogicBlockPanel = function (blockId) {
     CURRENT_BLOCK_ID = blockId;
     closeNodeForm();
-    document.getElementById('logicBlockPanelTitle').textContent = blockName || '?';
+    var block = FLOW_BLOCKS.find(function (b) { return b.id === blockId; });
+    document.getElementById('logicBlockPanelTitle').textContent = (block && block.name) || '?';
     panel.classList.remove('hidden');
     panel._fsToggle.enter();
     renderFlow();
@@ -2988,10 +2989,7 @@ function makeFloatPanelFullscreenable(panel, toggleBtn) {
   var requestedTab = {{ (request.args.get('tab') or '')|tojson }};
   if (requestedTab) switchBuilderTab(requestedTab);
   var requestedBlock = {{ request.args.get('block', type=int)|tojson }};
-  if (requestedBlock) {
-    var block = FLOW_BLOCKS.find(function (b) { return b.id === requestedBlock; });
-    if (block) openLogicBlockPanel(block.id, block.name);
-  }
+  if (requestedBlock) openLogicBlockPanel(requestedBlock);
 })();
 
 // ---------- Timeline d'animation : piste par élément, clips Animate.css ou