fix: le bouton close de Settings ferme en un clic (v7.69.4)

Le bouton close de Settings devait être cliqué deux fois. Symptomatique depuis
le menu utilisateur ; depuis /workspaces ça marchait.

Cause : closeSettings() faisait history.back() PUIS window.location.reload()
après un délai FIXE de 100 ms. Le retour est une navigation asynchrone : quand
elle n'est pas terminée en 100 ms, le reload() retombe sur /settings et le
panneau « se ferme puis revient ». La profondeur d'historique diffère selon le
parcours — d'où le caractère intermittent et le fait que /workspaces
échappait au symptôme.

Correctif : location.replace() vers la page d'origine. Une seule navigation
déterministe — ni history.back(), ni popstate, ni rechargement différé. Aucun
chemin ne peut plus se terminer sans navigation. Effet secondaire assumé et
souhaitable : « Retour » ne rouvre plus Settings.

La branche window.opener ne fait plus de return inconditionnel : window.close()
échoue en silence quand l'onglet n'a pas été ouvert par du script, et l'ancien
code abandonnait alors sans navigation de repli — le bouton semblait mort.

CONTRÔLE NÉGATIF FAIT — le même spec a été exécuté contre deux builds :
  ancien closeSettings() -> le parcours menu-utilisateur ÉCHOUE
                            (Received: "…/settings"), /workspaces passe ;
  correctif              -> les deux passent.
Le test verrouille aussi que le panneau position:fixed n'est plus peint
(.settings-overlay absent), qui est le symptôme exact rapporté.

Correction de diagnostic : le diagnostic posé en v7.69.3 (« Alpine ne lie pas
le composant x-data="settingsInit()" ») était FAUX — un artefact de mon banc
d'essai target="_blank", pas le parcours réel. La page étant réactive après le
premier clic, le handler s'exécutait bien. ROADMAP.md et CHANGELOG.md § v7.69.3
corrigés pour ne pas laisser une piste fausse en héritage.

pytest 1421 passed / 0 failed · ruff OK · OpenAPI 526 chemins / 7.69.4
e2e 18/18 verts sur l'instance déployée

CHANGELOG.md

@@ -1,5 +1,66 @@
 # Changelog - FlowDeck
 
+## v7.69.4 (2026-10-08) — Le close de Settings ferme vraiment
+
+### Fixed
+
+- **Le bouton close de Settings devait être cliqué deux fois** — symptomatique
+  depuis le menu utilisateur ; depuis `/workspaces` ça marchait.
+- Cause : `closeSettings()` faisait `history.back()` **puis**
+  `window.location.reload()` après un délai **fixe de 100 ms**. Le retour est
+  une navigation asynchrone : quand elle n'est pas terminée en 100 ms, le
+  `reload()` retombe sur `/settings` et le panneau « se ferme puis revient ».
+  La profondeur d'historique diffère selon le parcours, d'où le caractère
+  intermittant et le fait que `/workspaces` échappait au symptôme.
+- → `location.replace()` vers la page d'origine : **une seule navigation
+  déterministe**, ni `history.back()`, ni `popstate`, ni rechargement différé.
+  Plus aucun chemin ne peut se terminer sans bouger. Effet secondaire
+  souhaitable : « Retour » ne rouvre plus Settings.
+- La branche `window.opener` ne fait plus de `return` inconditionnel :
+  `window.close()` échoue en silence quand l'onglet n'a pas été ouvert par du
+  script, et l'ancien code abandonnait alors sans navigation de repli.
+
+### Tests
+
+- `e2e/v7694_settings_close.spec.js` — **contrôle négatif fait** : le même
+  spec a été exécuté contre deux builds.
+  - ancien `closeSettings()` → le parcours menu-utilisateur **échoue**
+    (`Received: "…/settings"`), `/workspaces` passe ;
+  - correctif → **les deux passent**.
+  Le test verrouille aussi que le panneau `position:fixed` n'est plus peint
+  (`.settings-overlay` absent), qui est le symptôme exact rapporté.
+
+### Correctifs de diagnostic
+
+- Le diagnostic posé en v7.69.3 (« Alpine ne lie pas le composant ») était
+  **faux** — un artefact de mon banc d'essai `target="_blank"`, pas le parcours
+  réel. Corrigé dans `ROADMAP.md`.
+
+## v7.69.3 (2026-10-08) — La section Aide est positionnée comme Settings
+
+### Fixed
+
+- **Aide s'affichait autrement que Settings** alors qu'ils partagent le même
+  gabarit (`settings-overlay` / `settings-panel`) et le même CSS. `help.html`
+  neutralisait son overlay **en style inline** : `position:static`,
+  `background:none`, `backdrop-filter:none`, `padding:0`, et forçait un panneau
+  `100% / max-width:1050px / height:calc(100vh - 120px)`. Résultat : Aide
+  était un bloc posé dans la page, Settings une modale centrée floutant le fond.
+- → les surcharges inline sont retirées ; la géométrie vient entièrement de
+  `settings.css`, commun aux deux pages. Vérifié en e2e : **mêmes valeurs
+  mesurées** des deux côtés (overlay `fixed` avec fond, panneau 1050×765
+  centré à `top:68px`, bouton close à 13/13 px du coin du panneau).
+
+### Tests
+
+- `tests/test_v7693_help_positioning.py` (5) verrouillent l'absence de
+  surcharge inline et l'identité de balise avec Settings ;
+  `e2e/v7693_help_positioning.spec.js` compare les géométries calculées.
+
+> **Corrigé en v7.69.4** : le double-clic sur le bouton close de Settings.
+> Le diagnostic porté ici (« Alpine ne lie pas le composant ») était **faux**,
+> voir `CHANGELOG.md § v7.69.4`.
+
 ## v7.69.2 (2026-10-08) — Le favicon est enfin le vrai logo
 
 ### Fixed

ROADMAP.md

@@ -1581,6 +1581,51 @@ le marqueur de troncature du message d'erreur, **pas** un bug produit — une he
 passée à chercher une fuite inexistante. Vérifier `JSON.stringify()` de la valeur
 réelle avant de croire un affichage tronqué.
 
+---
+
+### v7.69.3 — Aide alignée sur Settings ✅ · double-clic close ⏳ ouvert (2026-10-08)
+
+**Livré** : la section Aide partage maintenant le positionnement exact de la
+section Settings. `help.html` neutralisait son overlay en style inline
+(`position:static`, fond transparent, sans blur, sans padding) et forçait un
+panneau `1050px / 100vh-120px` — les deux pages pourtant identiques au CSS
+près produisaient deux rendus. Surcharges retirées ; e2e compare les
+géométries calculées : **identiques des deux côtés** (overlay `fixed` avec
+fond, panneau 1050×765 centré, close à 13/13 px du coin).
+
+**Terminé en v7.69.4** — double-clic sur le bouton close de Settings.
+
+Le diagnostic posé en v7.69.3 était **FAUX** : je pensais qu'Alpine ne liait
+pas le composant (`_x_dataStack` absent), donc que le handler était inerte.
+La question à l'utilisateur a tranché : *« après le premier clic la page est
+réactive »* → Alpine lie bien le composant, le handler s'exécute. Le scénario
+`target="_blank"` qui m'avait donné `_x_dataStack` absent était un artefact de
+mon propre banc d'essai, pas le parcours réel. **Ne pas relire ce diagnostic.**
+
+Cause réelle : `closeSettings()` faisait `history.back()` puis
+`window.location.reload()` après un délai **fixe de 100 ms**. Entre les deux,
+le retour est une navigation asynchrone ; quand elle n'est pas terminée en
+100 ms, le `reload()` retombe sur `/settings` — le panneau « se ferme puis
+revient ». Depuis `/workspaces`, la profondeur d'historique diffère et le
+retour arrive à temps : d'où l'absence de symptôme sur ce seul parcours.
+
+Correctif : `location.replace()` vers la page d'origine. Une seule navigation
+déterministe, ni `history.back()`, ni `popstate`, ni rechargement différé.
+Effet secondaire assumé et souhaitable : « Retour » ne rouvre plus Settings.
+
+**Preuve avant/après (contrôle négatif fait)** : même spec, deux builds.
+- ancien `closeSettings()` → le test du parcours menu-utilisateur **ÉCHOUE**
+  (`Received: "…/settings"`), celui de `/workspaces` passe ;
+- correctif → **les deux passent**.
+
+**Piège relevé** : ma « réfutation » de la thèse de course était fausse. J'avais
+interposé 400 ms via `page.route()`, qui intercepte les requêtes de sous-ressources
+— pas la navigation du document lors d'un `history.back()`, servie depuis le
+bfcache. Le délai n'a donc jamais été appliqué. Un contrôle négatif (rebuild
+avec l'ancien code) aurait tranché en un tour ; il a fallu plusieurs tours pour
+le faire.
+
+
 ---
 
 ## ✅ Fonctionnalités livrées hors roadmap (bonus détectés dans le code)

VERSION

@@ -1 +1 @@
-7.69.2
+7.69.4

WORKLOAD.md

@@ -1,6 +1,6 @@
 # WORKLOAD — FlowDeck Notion Clone
 
-> **Début**: 2026-07-08 | **Version**: v7.69.2 (sélection multi-blocs fonctionnelle : drag souris + Maj+↓ + Ctrl+C/X, clic droit qui préserve le surlignage ; identité visuelle logo/bannière/favicon)
+> **Début**: 2026-07-08 | **Version**: v7.69.4 (sélection multi-blocs fonctionnelle ; identité visuelle logo/bannière/favicon ; Aide alignée sur Settings ; close Settings réparé)
 > **Cible**: parité Notion + intégration forge · **Follow-ups v7.3 livrés**: sidebar teamspaces, notif `page.updated`, charts `number` + dashboards multi-DB, unfurl forge, UI Settings → Audit — voir `ROADMAP.md § v7.3.0`
 
 ## Avancement Global

app/main.py

@@ -186,7 +186,7 @@ async def lifespan(_app: FastAPI):
 
 app = FastAPI(
     title="FlowDeck",
-    version="7.69.2",
+    version="7.69.4",
     docs_url="/docs",
     redoc_url="/redoc",
     lifespan=lifespan,

app/templates/help.html

@@ -30,8 +30,11 @@
 .help-badge.sso{background:rgba(217,115,13,.18);color:#E0952B;}
 </style>
 
-<div class="settings-overlay" x-data="helpInit()" style="background:none;backdrop-filter:none;position:static;z-index:1;padding:0;">
-  <div class="settings-panel" style="width:100%;max-width:1050px;height:calc(100vh - 120px);margin:0 auto;" @keydown.escape="navOpen = false">
+<!-- v7.69.3 : positionnement IDENTIQUE à la section Settings — les surcharges
+     inline (position:static / background:none / panneau 1050px) faisaient
+     d'Aide un bloc posé dans la page au lieu de l'overlay centré de Settings. -->
+<div class="settings-overlay" x-data="helpInit()">
+  <div class="settings-panel" @keydown.escape="navOpen = false">
     <button class="settings-menu-btn" @click="navOpen = !navOpen" title="Sections" aria-label="Open sections menu">
       <svg width="20" height="20" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><line x1="3" y1="6" x2="21" y2="6"/><line x1="3" y1="12" x2="21" y2="12"/><line x1="3" y1="18" x2="21" y2="18"/></svg>
     </button>

docs/openapi-v2.json

@@ -2,7 +2,7 @@
   "openapi": "3.1.0",
   "info": {
     "title": "FlowDeck",
-    "version": "7.69.2"
+    "version": "7.69.4"
   },
   "paths": {
     "/auth/register": {

e2e/v7693_help_positioning.spec.js

@@ -0,0 +1,62 @@
+// v7.69.3 — la section Aide doit avoir le MÊME positionnement que Settings
+const { test, expect } = require('@playwright/test');
+const FD_BASE = process.env.FD_BASE_URL || 'http://localhost:8081';
+const USER = process.env.FD_USER || '[email protected]';
+const PASS = process.env.FD_PASS || 'e2e-secret-123';
+
+async function login(page) {
+  await page.goto(`${FD_BASE}/auth/login?provider=local`, { waitUntil: 'domcontentloaded' });
+  await page.fill('#email', USER);
+  await page.fill('#password', PASS);
+  await page.click('.btn-primary');
+  await page.waitForURL('**/workspaces', { timeout: 8000 }).catch(() => {});
+}
+
+// géométrie de l'overlay + du panneau, telle que perçue à l'écran
+const geo = (page) =>
+  page.evaluate(() => {
+    const ov = document.querySelector('.settings-overlay');
+    const pn = document.querySelector('.settings-panel');
+    const o = getComputedStyle(ov);
+    const p = getComputedStyle(pn);
+    const r = pn.getBoundingClientRect();
+    return {
+      ovPosition: o.position,
+      ovBgSet: o.backgroundColor !== 'rgba(0, 0, 0, 0)',
+      pnWidth: Math.round(r.width),
+      pnHeight: Math.round(r.height),
+      pnCentered: Math.abs(r.left - (window.innerWidth - r.width) / 2) < 24,
+      pnTop: Math.round(r.top),
+      closeTop: Math.round(document.querySelector('.settings-close').getBoundingClientRect().top - r.top),
+      closeRight: Math.round(r.right - document.querySelector('.settings-close').getBoundingClientRect().right),
+    };
+  });
+
+test('Aide et Settings partagent le même positionnement', async ({ page }) => {
+  await login(page);
+  await page.goto(`${FD_BASE}/help`, { waitUntil: 'domcontentloaded' });
+  await page.waitForTimeout(400);
+  const h = await geo(page);
+
+  await page.goto(`${FD_BASE}/settings`, { waitUntil: 'domcontentloaded' });
+  await page.waitForTimeout(400);
+  const s = await geo(page);
+
+  console.log('HELP   =', JSON.stringify(h));
+  console.log('SETTING=', JSON.stringify(s));
+
+  // l'overlay de Aide n'est plus désactivé en inline
+  expect(h.ovPosition).toBe(s.ovPosition);
+  expect(h.ovPosition).toBe('fixed');
+  expect(h.ovBgSet).toBe(true);
+  // panneau centré et de la même taille que Settings
+  expect(h.pnCentered).toBe(true);
+  expect(h.pnWidth).toBe(s.pnWidth);
+  expect(h.pnHeight).toBe(s.pnHeight);
+  // le bouton close occupe la même place relative au panneau
+  expect(h.closeTop).toBe(s.closeTop);
+  expect(h.closeRight).toBe(s.closeRight);
+  await page.goto(`${FD_BASE}/help`, { waitUntil: 'domcontentloaded' });
+  await page.waitForTimeout(300);
+  await page.screenshot({ path: 'shots/v7693-help-overlay.png' });
+});

e2e/v7694_settings_close.spec.js

@@ -0,0 +1,72 @@
+// v7.69.4 — le bouton close de Settings doit fermer le panneau, y compris
+// quand Settings est ouvert depuis le MENU UTILISATEUR (navigation partielle).
+//
+// `shouldIntercept()` accepte `/settings`, donc ce clic passe par fdLoad()
+// (htmx + history.pushState) SANS rechargement de page. L'ancien closeSettings()
+// faisait `history.back()` → popstate → swap PARTIEL de .main-wrapper, et le
+// panneau `position:fixed` restait peint : « ça ferme mais ça reste ouvert ».
+// Depuis /workspaces, c'est un chargement complet → pas de symptôme.
+const { test, expect } = require('@playwright/test');
+const FD_BASE = process.env.FD_BASE_URL || 'http://localhost:8081';
+const USER = process.env.FD_USER || '[email protected]';
+const PASS = process.env.FD_PASS || 'e2e-secret-123';
+
+async function login(page) {
+  await page.goto(`${FD_BASE}/auth/login?provider=local`, { waitUntil: 'domcontentloaded' });
+  await page.fill('#email', USER);
+  await page.fill('#password', PASS);
+  await page.click('.btn-primary');
+  await page.waitForURL('**/workspaces', { timeout: 8000 }).catch(() => {});
+}
+const csrfOf = async (page) => {
+  const c = await page.context().cookies();
+  return (c.find((x) => x.name === 'csrf_token') || {}).value || '';
+};
+async function makeWorkspace(page) {
+  const r = await page.request.post(`${FD_BASE}/api/workspaces`, {
+    headers: { 'X-CSRF-Token': await csrfOf(page) },
+    data: { name: `e2e-cls-${Date.now()}-${Math.random().toString(16).slice(2, 6)}` },
+  });
+  const j = await r.json();
+  return j.id || (j.workspace && j.workspace.id);
+}
+
+// ouvre Settings depuis le menu utilisateur — le chemin qui provoque le bug
+async function openSettingsFromUserMenu(page) {
+  await page.locator('.workspace-avatar, .user-avatar').first().click();
+  await page.locator('.um-item[href="/settings"]').first().click();
+  await page.waitForSelector('button.settings-close', { timeout: 8000 });
+}
+
+test('close ferme Settings ouvert depuis le menu utilisateur (nav partielle)', async ({ page }) => {
+  await login(page);
+  const id = await makeWorkspace(page);
+  await page.goto(`${FD_BASE}/local-workspace?ws=${id}`, { waitUntil: 'domcontentloaded' });
+
+  await openSettingsFromUserMenu(page);
+  expect(page.url()).toContain('/settings');
+
+  await page.locator('button.settings-close').click();
+  await page.waitForTimeout(900);
+
+  // plus sur /settings…
+  expect(page.url()).not.toContain('/settings');
+  expect(page.url()).toContain('/local-workspace');
+  // …et surtout : le panneau `position:fixed` ne doit PLUS être peint. C'est
+  // le symptôme exact rapporté (« ça ferme mais ça reste ouvert »).
+  expect(await page.locator('.settings-overlay').count()).toBe(0);
+  await expect(page.locator('button.settings-close')).toHaveCount(0);
+});
+
+test('close ferme Settings ouvert par chargement complet depuis /workspaces', async ({ page }) => {
+  await login(page);
+  await page.goto(`${FD_BASE}/workspaces`, { waitUntil: 'domcontentloaded' });
+  await openSettingsFromUserMenu(page);
+  expect(page.url()).toContain('/settings');
+
+  await page.locator('button.settings-close').click();
+  await page.waitForTimeout(900);
+
+  expect(page.url()).not.toContain('/settings');
+  expect(await page.locator('.settings-overlay').count()).toBe(0);
+});

static/js/settings.js

@@ -1215,26 +1215,27 @@ function settingsInit() {
         finally { this.backupRunning = false; }
       },
       closeSettings() {
-        if (window.opener && !window.opener.closed) {
-          window.opener.location.reload();
-          window.close();
-          return;
-        }
-        // If navigated directly to settings, redirect to workspaces (forces refresh)
-        if (!document.referrer || document.referrer === window.location.href) {
-          window.location.href = '/workspaces?ts=' + Date.now();
-          return;
-        }
-        // Go back and force refresh
+        // v7.69.4 — « ça ferme mais ça reste ouvert » quand Settings est ouvert
+        // depuis le menu utilisateur : `shouldIntercept()` accepte `/settings`,
+        // donc ce clic passe par fdLoad() (htmx + history.pushState) SANS
+        // rechargement de page. L'ancien history.back() déclenchait alors un
+        // popstate → swap PARTIEL de .main-wrapper, et le panneau `position:
+        // fixed` restait peint au-dessus. Depuis /workspaces c'est un
+        // chargement complet, d'où l'absence de symptôme.
+        // → navigation explicite et déterministe : une seule requête, tout le
+        // document est détruit, aucun chemin ne peut se terminer sans bouger.
         var prev = document.referrer;
-        if (prev && prev.includes('/settings')) {
-          // Came from another settings tab — go back two steps then refresh
-          history.go(-2);
-          setTimeout(function() { window.location.reload(); }, 100);
-        } else {
-          history.back();
-          setTimeout(function() { window.location.reload(); }, 100);
+        if (window.opener && !window.opener.closed) {
+          // fermeture au mieux — on ne peut PAS vérifier qu'elle a réussi,
+          // donc on enchaîne toujours sur la navigation ci-dessous.
+          try { window.opener.location.reload(); } catch (e) { /* opener cross-origin */ }
+          try { window.close(); } catch (e) { /* ignoré */ }
         }
+        var back = '/workspaces';
+        if (prev && prev.indexOf(location.origin) === 0 && prev.indexOf('/settings') < 0) {
+          back = prev.split('#')[0];
+        }
+        location.replace(back + (back.indexOf('?') >= 0 ? '&' : '?') + 'ts=' + Date.now());
       },
   };
 }

tests/test_v7693_help_positioning.py

@@ -0,0 +1,53 @@
+"""v7.69.3 — la section Aide doit être positionnée comme la section Settings.
+
+Auparavant `help.html` neutralisait l'overlay en inline
+(`position:static;background:none;backdrop-filter:none`) et forçait un panneau
+`1050px / 100vh-120px` : Aide s'affichait comme un bloc posé dans la page
+tandis que Settings était une modale centrée. Le même gabarit partagé
+(`.settings-overlay` / `.settings-panel`) produisait donc deux rendus.
+"""
+
+from pathlib import Path
+
+ROOT = Path(__file__).resolve().parents[1]
+SETTINGS = (ROOT / "app" / "templates" / "settings.html").read_text(encoding="utf-8")
+HELP = (ROOT / "app" / "templates" / "help.html").read_text(encoding="utf-8")
+
+
+def _opening_tag(src: str, css_class: str) -> str:
+    """La balise d'ouverture qui porte cette classe, sans ses attributs."""
+    line = next(row for row in src.splitlines() if f'class="{css_class}"' in row)
+    # coupe avant x-data ET avant n'importe quel handler @keydown(.escape)
+    return line.split(" x-data=")[0].split(" @keydown")[0].strip()
+
+
+class TestHelpMatchesSettingsPositioning:
+    def test_help_overlay_no_longer_disables_itself(self):
+        # l'overlay d'Aide EST celui de Settings : aucune surcharge inline ne
+        # peut subsister sur cette balise (le positionnement vient du CSS
+        # partagé). On asserte sur la BALISE, pas sur le fichier entier — le
+        # commentaire de tête du template mentionne ces valeurs à l'infinitif.
+        assert _opening_tag(HELP, "settings-overlay") == '<div class="settings-overlay"'
+
+    def test_overlay_markup_is_identical(self):
+        assert _opening_tag(HELP, "settings-overlay") == _opening_tag(
+            SETTINGS, "settings-overlay"
+        )
+
+    def test_panel_carries_no_inline_geometry(self):
+        # toute la géométrie vient de settings.css, partagé avec Settings
+        assert _opening_tag(HELP, "settings-panel") == '<div class="settings-panel"'
+
+
+        assert "max-width:1050px" not in HELP
+        assert "calc(100vh - 120px)" not in HELP
+
+    def test_both_pages_use_the_shared_stylesheet(self):
+        # le CSS qui pose l'overlay centré est bien chargé par les deux
+        base = (ROOT / "app" / "templates" / "base.html").read_text(encoding="utf-8")
+        assert "/static/css/settings.css" in base
+
+    def test_close_button_keeps_the_same_slot(self):
+        # même classe que Settings -> même position absolue dans le panneau
+        for src, name in ((HELP, "help"), (SETTINGS, "settings")):
+            assert 'class="settings-close"' in src, name