@artymclabin/qa-review 0.3.9 → 0.3.10

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## 0.3.10 - 2026-09-19
+
+- OUTSIDE TAP MINIMIZES THE PANEL, ON MOBILE: the only way to collapse the
+  card used to be the small minimize button in its header, and hitting a
+  small target on a phone is real friction. A pointerdown anywhere outside
+  the card now minimizes it too, scoped to mobile-width viewports (desktop
+  keeps the explicit button only, since a stray click outside a small
+  floating panel on a mouse is far more likely incidental than intentional).
+  Uses a capture-phase listener plus a contains() check against the card, so
+  dragging the card by its header (which also starts with a pointerdown,
+  from inside the card) is unaffected.
+
+## 0.3.9 - 2026-09-13
+
+- PICKED FORM CONTROLS ARE NAMED BY LABEL, NOT BY TAG: a form control has no
+  textContent, so the element picker fell through to `tagName.toLowerCase()`
+  for every input, textarea and select. Picking the seven link fields on the
+  GameReady pitch page produced seven identical `<input>` references, with
+  no way for the reviewer to say which row they meant. The picker now reads
+  the name a human would: own text, accessible name (aria-label /
+  aria-labelledby), the associated `<label>` (including a wrapping label),
+  placeholder, alt, title, current value, name attribute, and only then the
+  tag.
+
 ## 0.3.8 - 2026-08-09
 
 - BACK TO REVIEW IS AN UNDO, NOT A RESTART: the finish panel's "Back to review"

package/dist/client/QAReviewOverlay.js

@@ -291,6 +291,31 @@ export function QAReviewOverlay({ items, target, gateParam, stateUrl, submitUrl,
         mq.addEventListener("change", onChange);
         return () => mq.removeEventListener("change", onChange);
     }, []);
+    // OUTSIDE TAP TO MINIMIZE (mobile only): pixel-hunting for the small
+    // minimize button is real friction on a phone, so a tap anywhere outside
+    // the card collapses it to the bubble - same result as the button, far
+    // cheaper to hit. Scoped to mobile width: on desktop a stray click outside
+    // a small floating panel is much more likely incidental than intentional.
+    // Capture phase + native listener so it fires ahead of (and regardless of)
+    // any stopPropagation on the underlying page, and a pointerdown that starts
+    // INSIDE the card (e.g. dragging the header) is excluded by the contains()
+    // check, so dragging is unaffected.
+    React.useEffect(() => {
+        if (typeof window.matchMedia !== "function")
+            return;
+        if (!active || finished || minimized || mobilePreview)
+            return;
+        const onPointerDown = (e) => {
+            if (!window.matchMedia(`(max-width: ${AUTO_BUBBLE_MAX_WIDTH_PX}px)`).matches)
+                return;
+            const target = e.target;
+            if (cardRef.current && target && !cardRef.current.contains(target)) {
+                setMinimized(true);
+            }
+        };
+        document.addEventListener("pointerdown", onPointerDown, true);
+        return () => document.removeEventListener("pointerdown", onPointerDown, true);
+    }, [active, finished, minimized, mobilePreview]);
     // Hydrate verdicts and LAND on the first outstanding item. The DURABLE
     // server ledger is the source of truth; localStorage is only a fast cache.
     // 🚨 The store NEVER pre-approves (no seed). The round derives from

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@artymclabin/qa-review",
-  "version": "0.3.9",
+  "version": "0.3.10",
   "description": "Interactive on-page QA review overlay (spotlight walkthrough, approve/reject verdict ledger) with a framework-agnostic server handler factory and a self-provisioning Postgres store.",
   "license": "MIT",
   "repository": {