mcdview 0.28.0__tar.gz → 0.28.2__tar.gz

{mcdview-0.28.0 → mcdview-0.28.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcdview
-Version: 0.28.0
+Version: 0.28.2
 Summary: Turn a SQL data model into one self-contained, interactive HTML page
 Project-URL: Homepage, https://github.com/Gheop/mcdview
 Project-URL: Demo, https://gheop.github.io/mcdview/
@@ -324,6 +324,26 @@ file's header.
 
 ## Changelog
 
+### v0.28.2 — Fix the theme toggle and a stuck audit-FK checkbox (2026-09-01)
+
+- The theme button no longer needs two clicks to reach dark from a light OS. It
+  cycled system → light → dark, but "system" and "light" look identical on a
+  light OS, so the first click seemed to do nothing. A click now always changes
+  the visible theme, skipping any step that would look the same.
+- The "audit FKs" checkbox is hidden again when a model has no audit FKs. A
+  `#hud label { display: flex }` rule overrode the `hidden` attribute, so the
+  checkbox showed even with nothing to toggle.
+
+### v0.28.1 — Fix links pointing into space when saved positions load (2026-08-31)
+
+- A viewer who had dragged tables saw the FK links briefly point into empty
+  space on the next open. Restoring the saved layout re-applied each box's
+  position after the page had settled, so the boxes animated from their
+  auto-placement spot to their saved spot and the first links were drawn against
+  them mid-slide. The restore now jumps the boxes into place with no animation,
+  so the links are correct from the first paint. Added a headless-Chrome
+  regression test (`tests/test_positions.py`).
+
 ### v0.28.0 — Auto attribution badge on branded pages (2026-08-31)
 
 - A branded page (`--logo` or `--home-url`) now carries a discreet clickable

{mcdview-0.28.0 → mcdview-0.28.2}/README.md

@@ -302,6 +302,26 @@ file's header.
 
 ## Changelog
 
+### v0.28.2 — Fix the theme toggle and a stuck audit-FK checkbox (2026-09-01)
+
+- The theme button no longer needs two clicks to reach dark from a light OS. It
+  cycled system → light → dark, but "system" and "light" look identical on a
+  light OS, so the first click seemed to do nothing. A click now always changes
+  the visible theme, skipping any step that would look the same.
+- The "audit FKs" checkbox is hidden again when a model has no audit FKs. A
+  `#hud label { display: flex }` rule overrode the `hidden` attribute, so the
+  checkbox showed even with nothing to toggle.
+
+### v0.28.1 — Fix links pointing into space when saved positions load (2026-08-31)
+
+- A viewer who had dragged tables saw the FK links briefly point into empty
+  space on the next open. Restoring the saved layout re-applied each box's
+  position after the page had settled, so the boxes animated from their
+  auto-placement spot to their saved spot and the first links were drawn against
+  them mid-slide. The restore now jumps the boxes into place with no animation,
+  so the links are correct from the first paint. Added a headless-Chrome
+  regression test (`tests/test_positions.py`).
+
 ### v0.28.0 — Auto attribution badge on branded pages (2026-08-31)
 
 - A branded page (`--logo` or `--home-url`) now carries a discreet clickable

{mcdview-0.28.0 → mcdview-0.28.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "mcdview"
-version = "0.28.0"
+version = "0.28.2"
 description = "Turn a SQL data model into one self-contained, interactive HTML page"
 readme = "README.md"
 requires-python = ">=3.9"

{mcdview-0.28.0 → mcdview-0.28.2}/templates/explorateur.html

@@ -46,6 +46,10 @@
     box-shadow: 0 2px 12px #0b74c455; }
   .table.efface { opacity: 0; pointer-events: none; }
   .table.deplace { transition: none; cursor: grabbing; }
+  /* restoring saved positions must be instant, never an animated slide from the
+     auto-placement spot (the initial links would be drawn against the moving box
+     and point into space until a redraw); see restaurerPositions */
+  #plan.instant .table { transition: none; }
   .table h3 { margin: 0; padding: 4px 10px; font-size: 13px; color: #1c1e22;
     border-radius: 4px 4px 0 0; white-space: nowrap; }
   .table ul { margin: 0; padding: 3px 10px 5px; list-style: none; }
@@ -151,6 +155,7 @@
   #hud button { padding: 4px 10px; border: 1px solid var(--bord); border-radius: 5px;
     background: var(--panneau); color: var(--encre); cursor: pointer; }
   #hud label { display: flex; gap: 4px; align-items: center; font-size: 13px; }
+  #hud label[hidden] { display: none; }  /* author display:flex beats [hidden]; restore it */
   #compte { font-size: 12px; opacity: .55; white-space: nowrap; }
   .table:hover { border-color: var(--ligne-vive); }
   #panneau { position: absolute; top: 0; right: 0; bottom: 0; width: 340px;
@@ -392,6 +397,12 @@ function sauverPositions() {
   let pos = null;
   try { pos = JSON.parse(localStorage.getItem(CLE_POS)); } catch (e) { return; }
   if (!pos) return;
+  // the position snapshot above forced a layout at the auto-placement spot, so
+  // changing left/top here would animate (the .table left/top transition). Kill
+  // the transition while we jump the boxes to their saved spots, flush the
+  // layout, then re-enable it — otherwise the boxes slide in and the first links
+  // are drawn against them mid-flight, pointing into space until the next redraw.
+  plan.classList.add("instant");
   Object.keys(elems).forEach(cle => {
     if (pos[cle] && typeof pos[cle].x === "number") {
       origines[cle] = pos[cle];
@@ -399,6 +410,8 @@ function sauverPositions() {
       elems[cle].style.top = pos[cle].y + "px";
     }
   });
+  plan.offsetLeft;  // flush the jump so it never animates
+  plan.classList.remove("instant");
 })();
 
 // --- schema legend: one chip per schema, click to frame its zone ---
@@ -741,7 +754,10 @@ document.addEventListener("keydown", e => {
   else if (e.key === "r" && !btnReorg.disabled) reorganiser();
 });
 
-// theme toggle: cycle system → light → dark, remembered per viewer
+// theme toggle: cycle system → light → dark, remembered per viewer. A click
+// always changes the visible theme: when the next state would render the same
+// as now (e.g. "system" equals "light" on a light OS), skip it — otherwise the
+// first click after load looked like a no-op (system → light on a light OS)
 const btnTheme = document.getElementById("theme");
 (function initTheme() {
   // a ?theme= deep-link wins over the remembered choice (see view deep-links)
@@ -753,8 +769,12 @@ const btnTheme = document.getElementById("theme");
 })();
 btnTheme.addEventListener("click", () => {
   const ordre = ["", "light", "dark"];
+  const sysSombre = matchMedia("(prefers-color-scheme: dark)").matches;
+  const rendu = e => e || (sysSombre ? "dark" : "light");  // what a state looks like
   const actuel = document.documentElement.dataset.theme || "";
-  const suivant = ordre[(ordre.indexOf(actuel) + 1) % ordre.length];
+  let i = ordre.indexOf(actuel), suivant;
+  do { i = (i + 1) % ordre.length; suivant = ordre[i]; }
+  while (rendu(suivant) === rendu(actuel));  // never a step that looks identical
   if (suivant) document.documentElement.dataset.theme = suivant;
   else delete document.documentElement.dataset.theme;
   try {