From aece8c55268eb9393d7c8cf2168789648363fa09 Mon Sep 17 00:00:00 2001
From: s-FunIA <macstudio@an-ArcraT-IA.local>
Date: Mon, 19 Jan 2026 11:31:32 +0100
Subject: [PATCH] ui: propose modal 2-steps + clean local urls

---
 src/components/ProposeModal.astro | 147 ++++++++++++++++++++++++------
 src/layouts/EditionLayout.astro   |  45 +++++++--
 2 files changed, 153 insertions(+), 39 deletions(-)

diff --git a/src/components/ProposeModal.astro b/src/components/ProposeModal.astro
index 47d323b..a2481c8 100644
--- a/src/components/ProposeModal.astro
+++ b/src/components/ProposeModal.astro
@@ -1,41 +1,88 @@
-<dialog id="propose-modal" aria-label="Qualifier la proposition">
+<dialog id="propose-modal" aria-label="Qualifier la proposition" data-step="1">
   <form method="dialog" class="box">
     <header class="hd">
       <h2>Qualifier la proposition</h2>
       <button value="cancel" class="x" aria-label="Fermer">✕</button>
     </header>
 
-    <p class="sub">
-      Optionnel : choisis une intention (pour tri & traitement éditorial). Sinon, continue sans préciser.
-    </p>
+    <!-- STEP 1 -->
+    <section class="step step-1" aria-label="Choisir le type">
+      <p class="sub">
+        D’abord : quel type de ticket veux-tu ouvrir ?
+      </p>
 
-    <div class="grid">
-      <button type="button" data-category="cat/style">Style / lisibilité</button>
-      <button type="button" data-category="cat/lexique">Lexique / terminologie</button>
-      <button type="button" data-category="cat/argument">Argument / structure</button>
-      <button type="button" data-category="cat/redondance">Redondance</button>
-      <button type="button" data-category="cat/source">Source / vérification</button>
-      <button type="button" data-category="">Continuer sans préciser</button>
-    </div>
+      <div class="grid">
+        <button type="button" data-kind="correction">✍️ Correction (rédaction / clarté)</button>
+        <button type="button" data-kind="fact">🔎 Vérification factuelle / sources</button>
+      </div>
 
-    <footer class="ft">
-      <small>Astuce : touche <kbd>Esc</kbd> pour fermer.</small>
-    </footer>
+      <footer class="ft">
+        <small>Étape 1/2 — <kbd>Esc</kbd> pour fermer.</small>
+      </footer>
+    </section>
+
+    <!-- STEP 2 -->
+    <section class="step step-2" aria-label="Choisir la catégorie">
+      <p class="sub">
+        Optionnel : choisis une intention (pour tri & traitement éditorial). Sinon, continue sans préciser.
+      </p>
+
+      <div class="grid">
+        <button type="button" data-category="cat/style">Style / lisibilité</button>
+        <button type="button" data-category="cat/lexique">Lexique / terminologie</button>
+        <button type="button" data-category="cat/argument">Argument / structure</button>
+        <button type="button" data-category="cat/redondance">Redondance</button>
+        <button type="button" data-category="cat/source">Source / vérification</button>
+        <button type="button" data-category="">Continuer sans préciser</button>
+      </div>
+
+      <footer class="ft row">
+        <button type="button" class="back" data-back="1">← Retour</button>
+        <small>Étape 2/2 — <kbd>Esc</kbd> pour fermer.</small>
+      </footer>
+    </section>
   </form>
 </dialog>
 
 <style>
-  dialog { border: none; padding: 0; border-radius: 16px; width: min(720px, calc(100vw - 2rem)); }
+  dialog { border: none; padding: 0; border-radius: 18px; width: min(760px, calc(100vw - 2rem)); }
   dialog::backdrop { background: rgba(0,0,0,.55); }
-  .box { padding: 1rem 1rem 0.75rem; }
+
+  .box { padding: 1rem 1rem .85rem; }
   .hd { display:flex; align-items:center; justify-content:space-between; gap:.75rem; }
   .hd h2 { margin:0; font-size:1.1rem; }
   .x { border:0; background:transparent; font-size:1.1rem; cursor:pointer; }
-  .sub { margin:.5rem 0 1rem; opacity:.9; }
-  .grid { display:grid; grid-template-columns: repeat(auto-fit, minmax(220px, 1fr)); gap:.5rem; }
-  .grid button { padding: .75rem .8rem; border-radius: 12px; border: 1px solid rgba(0,0,0,.18); background: rgba(255,255,255,.96); color: #111; cursor: pointer; text-align: left; }
+
+  .sub { margin:.55rem 0 1rem; opacity:.92; line-height: 1.35; }
+
+  .grid { display:grid; grid-template-columns: repeat(auto-fit, minmax(240px, 1fr)); gap:.55rem; }
+  .grid button {
+    padding: .8rem .9rem;
+    border-radius: 14px;
+    border: 1px solid rgba(0,0,0,.18);
+    background: rgba(255,255,255,.96);
+    color: #111;
+    cursor: pointer;
+    text-align: left;
+    line-height: 1.2;
+  }
   .grid button:hover { background: #fff; }
+
+  .ft { margin-top:.85rem; opacity:.88; }
+  .ft.row { display:flex; align-items:center; justify-content:space-between; gap:1rem; }
+  .back {
+    border: 1px solid rgba(0,0,0,.18);
+    background: rgba(255,255,255,.96);
+    color:#111;
+    border-radius: 12px;
+    padding: .55rem .7rem;
+    cursor: pointer;
+  }
   kbd { padding:0 .35rem; border:1px solid rgba(0,0,0,.2); border-radius:6px; }
+
+  .step { display:none; }
+  dialog[data-step="1"] .step-1 { display:block; }
+  dialog[data-step="2"] .step-2 { display:block; }
 </style>
 
 <script is:inline>
@@ -45,48 +92,88 @@
 
   /** @type {URL|null} */
   let pending = null;
+  /** @type {"correction"|"fact"|""} */
+  let kind = "";
 
   const upsertLine = (text, key, value) => {
-    const re = new RegExp(`^${key}:\\s*.*$`, "mi");
-    // Si "continuer sans préciser" : on ne touche pas au body
-    if (!value) return text;
+    const re = new RegExp(`^\\s*${key}\\s*:\\s*.*$`, "mi");
     if (re.test(text)) return text.replace(re, `${key}: ${value}`);
     const sep = text && !text.endsWith("\n") ? "\n" : "";
     return text + sep + `${key}: ${value}\n`;
   };
 
-  const openModalFor = (url) => {
+  const setTitlePrefix = (u, prefix) => {
+    const t = u.searchParams.get("title") || "";
+    const cleaned = t.replace(/^\[[^\]]+\]\s*/,"");
+    u.searchParams.set("title", `[${prefix}] ${cleaned}`.trim());
+  };
+
+  const openWith = (url) => {
     pending = url;
+    kind = "";
+    dlg.dataset.step = "1";
     if (typeof dlg.showModal === "function") dlg.showModal();
     else window.open(url.toString(), "_blank", "noopener,noreferrer");
   };
 
-  // Interception des clics sur "Proposer"
+  // Intercepte UNIQUEMENT les liens marqués data-propose
   document.addEventListener("click", (e) => {
     const a = e.target?.closest?.("a[data-propose]");
     if (!a) return;
 
     e.preventDefault();
 
-    // URL réelle stockée par EditionLayout (propose.dataset.url)
+    // L'URL réelle est dans data-url (préparée côté EditionLayout)
     const rawUrl = a.dataset.url || a.getAttribute("href") || "";
     if (!rawUrl || rawUrl === "#") return;
 
     try {
-      openModalFor(new URL(rawUrl, window.location.origin));
+      openWith(new URL(rawUrl));
     } catch {
       window.open(rawUrl, "_blank", "noopener,noreferrer");
     }
   });
 
-  // Choix d’une catégorie -> injection "Category:" + ouverture Gitea
+  // Step 1: type
+  dlg.addEventListener("click", (e) => {
+    const btn = e.target?.closest?.("button[data-kind]");
+    if (!btn || !pending) return;
+
+    kind = btn.getAttribute("data-kind") || "";
+
+    let body = pending.searchParams.get("body") || "";
+
+    if (kind === "fact") {
+      body = upsertLine(body, "Type", "type/fact-check");
+      body = upsertLine(body, "State", "state/a-sourcer");
+      setTitlePrefix(pending, "Fact-check");
+    } else {
+      body = upsertLine(body, "Type", "type/correction");
+      body = upsertLine(body, "State", "state/recevable");
+      setTitlePrefix(pending, "Correction");
+    }
+
+    pending.searchParams.set("body", body);
+    dlg.dataset.step = "2";
+  });
+
+  // Back
+  dlg.addEventListener("click", (e) => {
+    const back = e.target?.closest?.("button[data-back]");
+    if (!back) return;
+    dlg.dataset.step = "1";
+  });
+
+  // Step 2: category + open
   dlg.addEventListener("click", (e) => {
     const btn = e.target?.closest?.("button[data-category]");
     if (!btn || !pending) return;
 
     const cat = btn.getAttribute("data-category") || "";
-    const body = pending.searchParams.get("body") || "";
-    pending.searchParams.set("body", upsertLine(body, "Category", cat));
+    let body = pending.searchParams.get("body") || "";
+    if (cat) body = upsertLine(body, "Category", cat);
+
+    pending.searchParams.set("body", body);
 
     dlg.close();
     window.open(pending.toString(), "_blank", "noopener,noreferrer");
diff --git a/src/layouts/EditionLayout.astro b/src/layouts/EditionLayout.astro
index 0095571..d9addae 100644
--- a/src/layouts/EditionLayout.astro
+++ b/src/layouts/EditionLayout.astro
@@ -70,6 +70,11 @@ const GITEA_REPO = import.meta.env.PUBLIC_GITEA_REPO ?? "";
 <!-- IMPORTANT: define:vars injecte les constantes dans le JS sans templating fragile -->
     <script is:inline define:vars={{ GITEA_BASE, GITEA_OWNER, GITEA_REPO }}>
 (() => {
+  // Nettoyage si un ancien bug a injecté ?body=... dans l'URL
+  if (window.location.search.includes("body=")) {
+    history.replaceState(null, "", window.location.pathname + window.location.hash);
+  }
+
   const title = document.body.dataset.docTitle || document.title;
   const version = document.body.dataset.docVersion || "";
 
@@ -79,7 +84,11 @@ const GITEA_REPO = import.meta.env.PUBLIC_GITEA_REPO ?? "";
     const base = String(GITEA_BASE).replace(/\/+$/, "");
     const issue = new URL(`${base}/${GITEA_OWNER}/${GITEA_REPO}/issues/new`);
 
-    const local = new URL(window.location.href);
+    // URL locale "propre" : on ignore totalement query-string (?body=...)
+    const local = new URL(window.location.origin + window.location.pathname);
+    // évite d’embarquer des paramètres parasites (ex: ?body=... issus de tests)
+    local.searchParams.delete("body");
+    local.searchParams.delete("title");
     local.hash = anchorId;
     const path = local.pathname;
 
@@ -127,7 +136,7 @@ const GITEA_REPO = import.meta.env.PUBLIC_GITEA_REPO ?? "";
     citeBtn.textContent = "Citer";
 
     citeBtn.addEventListener("click", async () => {
-      const url = new URL(window.location.href);
+      const url = new URL(window.location.origin + window.location.pathname);
       url.hash = p.id;
       const cite = `${title}${version ? ` (v${version})` : ""} — ${url.toString()}`;
 
@@ -145,16 +154,34 @@ const GITEA_REPO = import.meta.env.PUBLIC_GITEA_REPO ?? "";
     tools.appendChild(citeBtn);
 
     if (giteaReady) {
-      const propose = document.createElement("a");
-      propose.className = "para-propose";
-      propose.href = "#";
-      propose.textContent = "Proposer";
-      propose.setAttribute("aria-label", "Proposer une correction sur Gitea");
+  const propose = document.createElement("a");
+  propose.className = "para-propose";
+  propose.href = "#";
+  propose.textContent = "Proposer";
+  propose.setAttribute("aria-label", "Proposer une correction sur Gitea");
+  propose.setAttribute("data-propose","1");
 
-      propose.setAttribute("data-propose","1");
+  propose.addEventListener("click", (e) => {
+    e.preventDefault();
+    const raw = (p.textContent || "").trim().replace(/\s+/g, " ");
+    const excerpt = raw.length > 420 ? (raw.slice(0, 420) + "…") : raw;
+    const url = buildIssueURL(p.id, excerpt);
 
-      tools.appendChild(propose);
+    // on stocke l'URL pour le modal
+    propose.dataset.url = url.toString();
+
+    // fallback: si le modal n'existe pas -> ouverture directe
+    const dlg = document.getElementById("propose-modal");
+    if (!dlg || typeof dlg.showModal !== "function") {
+      window.open(propose.dataset.url, "_blank", "noopener,noreferrer");
+      return;
     }
+    // si modal existe, il va intercepter le click globalement
+    // donc on ne fait rien ici de plus.
+  });
+
+  tools.appendChild(propose);
+}
 
     p.appendChild(tools);
   }