@fusion.dev/helix-core 0.179.0 → 0.180.1

package/custom-elements.json

@@ -378,6 +378,15 @@
               "default": "'inline'",
               "description": "Which axis the readings are laid out along.\n\n**An axis and not a boolean, because two of the four charts do not use x.**\nA horizontal bar chart runs its categories *down* the box, so resolving\nevery reading from the pointer's x leaves a move from the first row to the\nlast never changing the answer: measured on four rows, all four reported\nthe third, which is the slot the middle of the box falls in. It looks like\na tooltip that will not follow and is a tooltip reading the wrong axis.\n\n`none` is the ring. Its slices are real elements at every angle, so the\nreading under the pointer is the one the pointer is over, and arithmetic on\neither axis would answer with whichever slice shared a coordinate: at three\no'clock and at nine o'clock, x alone gives the same slice for opposite\nsides. The slices call `set` and this only holds the answer."
             },
+            {
+              "kind": "field",
+              "name": "drawnLtr",
+              "type": {
+                "text": "boolean"
+              },
+              "default": "false",
+              "description": "Whether the plot is drawn left to right on every page, as Scatter Chart's\nplane is, so the arrows follow the drawing rather than the page's\ndirection. Off, the arrow pointing along the line of text moves on."
+            },
             {
               "kind": "field",
               "name": "onReading",
@@ -2530,7 +2539,7 @@
       "declarations": [
         {
           "kind": "class",
-          "description": "Passes a click on the host itself to the control inside that answers clicks.\n\n**A click dispatched on the host reaches nothing that toggles.** A switch,\na checkbox or a radio renders its control in its shadow root, and only a\nclick that lands there moves it. `element.click()`, a test's or an\nautomation's, is dispatched on the host, and so is the click a `<label for>`\non the page sends the element it names: `helix-switch` left `checked` false\nunder `click()` while a pointer press on it turned it on. A native checkbox\ntoggles on both, and code written against one expects it.\n\n**Only a click that starts at the host**, whose path begins there. A press\ninside the shadow root reaches the host too, retargeted, and has already\ndone its work where it landed; the click this sends to the control reaches\nthe host the same way, and goes no further.",
+          "description": "Passes a click on the host itself to the control inside that answers clicks.\n\n**A click dispatched on the host reaches nothing that toggles.** A switch,\na checkbox or a radio renders its control in its shadow root, and only a\nclick that lands there moves it. `element.click()`, a test's or an\nautomation's, is dispatched on the host, and so is the click a `<label for>`\non the page sends the element it names: `helix-switch` left `checked` false\nunder `click()` while a pointer press on it turned it on. A native checkbox\ntoggles on both, and code written against one expects it.\n\n**One click reaches the page for each press, as from a native control.**\nThe control's click used to be sent composed, so it crossed the shadow root\nand reached the host after the host's own: a listener on the host heard two\nclicks for one `click()` or one press on a `<label for>`, where one on a\nnative checkbox hears one. So there are two paths, and each leaves exactly\none click outside the shadow root:\n\n- **The host's `click()`** calls {@link click}, which clicks the control\n  instead. That click is the only one: it reaches the host retargeted, as a\n  click inside a native control does, after the control has moved, so a\n  listener reads the new state and `preventDefault()` puts the old one back.\n- **A click that starts at the host** some other way, from a `<label for>`,\n  a press on the host's own box, or `dispatchEvent`, is the click the page\n  hears. The control is sent a copy that is not composed, so it moves the\n  control and stops at the shadow root. The copy goes in the capture phase,\n  so the host's own listeners already read the new state, as a native\n  checkbox's do. A press inside the shadow root has already landed on its\n  control and is left alone.",
           "name": "HostClickController",
           "members": [
             {
@@ -2551,6 +2560,16 @@
                 }
               }
             },
+            {
+              "kind": "method",
+              "name": "click",
+              "return": {
+                "type": {
+                  "text": "boolean"
+                }
+              },
+              "description": "Clicks the control, for the host's own `click()`. Answers `false` when\nthere is no control yet, before the first render, so the host can fall\nback to its own click."
+            },
             {
               "kind": "field",
               "name": "forward",
@@ -12852,6 +12871,14 @@
       "path": "src/index.ts",
       "declarations": [],
       "exports": [
+        {
+          "kind": "js",
+          "name": "auditChildren",
+          "declaration": {
+            "name": "auditChildren",
+            "package": "#audit-children"
+          }
+        },
         {
           "kind": "js",
           "name": "auditName",
@@ -24587,6 +24614,86 @@
         }
       ]
     },
+    {
+      "kind": "javascript-module",
+      "path": "src/utils/audit-children/audit-children-off.ts",
+      "declarations": [
+        {
+          "kind": "function",
+          "name": "auditChildren",
+          "description": "The unregistered-child warning in a production build: nothing.\n\n`#audit-children` resolves here unless the bundler or runtime asks for the\n`development` condition, so a production build carries none of the audit's\ncode. The signature is the audit's, so a call site reads the same in both\nbuilds. See `audit-children.ts`.",
+          "type": {
+            "text": "typeof audit"
+          }
+        }
+      ],
+      "exports": [
+        {
+          "kind": "js",
+          "name": "auditChildren",
+          "declaration": {
+            "name": "auditChildren",
+            "module": "src/utils/audit-children/audit-children-off.ts"
+          }
+        }
+      ]
+    },
+    {
+      "kind": "javascript-module",
+      "path": "src/utils/audit-children/audit-children.ts",
+      "declarations": [
+        {
+          "kind": "function",
+          "name": "auditChildren",
+          "return": {
+            "type": {
+              "text": "void"
+            }
+          },
+          "parameters": [
+            {
+              "name": "host",
+              "type": {
+                "text": "HTMLElement"
+              }
+            },
+            {
+              "name": "part",
+              "type": {
+                "text": "string"
+              }
+            },
+            {
+              "name": "entry",
+              "type": {
+                "text": "string"
+              }
+            },
+            {
+              "name": "consequence",
+              "type": {
+                "text": "string"
+              }
+            },
+            {
+              "name": "drawn",
+              "default": "false"
+            }
+          ],
+          "description": "Reports a child from another family that a container holds and nobody\nregistered, naming the entry to import.\n\n**A container does not register another family's children.** An option\nbelongs to the Option family, and a page writing one inside a Select imports\n`@fusion.dev/helix/option` itself, as it imports the Select. Left out, the\noption is an unknown element: the list reads no choices and nothing throws.\nThis says so, once for each container tag and child tag on the page, with\nthe line to add.\n\nAsked once the page has loaded and painted, so an entry a later module\nimports is not taken for a missing one.\n\n**In a development build only**, as `missing-name` is: every container would\npay for a sentence meant for the person writing the page. Core imports it as\n`#audit-children`, which `package.json` resolves here under the\n`development` condition and to `audit-children-off.ts`, which does nothing,\neverywhere else.\n\n`part` is the child's tag after the tier, such as `option`, and `entry` its\nfamily's entry, such as `option`; the host's `tier` gives the rest of both.\n`consequence` says what the missing child costs, after \"so\". `drawn` is for\nchildren the host draws from data, such as the options of Listbox's `items`:\nthe page wrote none, so an undefined tag is reported without one in the\nhost's light DOM."
+        }
+      ],
+      "exports": [
+        {
+          "kind": "js",
+          "name": "auditChildren",
+          "declaration": {
+            "name": "auditChildren",
+            "module": "src/utils/audit-children/audit-children.ts"
+          }
+        }
+      ]
+    },
     {
       "kind": "javascript-module",
       "path": "src/utils/audit-name/audit-name-off.ts",
@@ -33987,7 +34094,7 @@
           "type": {
             "text": "string"
           },
-          "default": "'0.179.0'",
+          "default": "'0.180.1'",
           "description": "The version of the package this file ships in.\n\n**Written by `fusion/scripts/version.mjs`** wherever it writes the package's\n`package.json`, and stamped back into the repository beside it after a\nrelease, so the two never disagree; a spec beside this holds them to it.\nNothing else in the running code knows which release it is."
         }
       ],
@@ -34623,6 +34730,6 @@
   ],
   "helix": {
     "package": "@fusion.dev/helix-core",
-    "version": "0.179.0"
+    "version": "0.180.1"
   }
 }

package/dist/controllers/chart-pointer-controller/chart-pointer-controller.d.ts

@@ -162,6 +162,12 @@ export declare class ChartPointerController implements ReactiveController {
      * sides. The slices call `set` and this only holds the answer.
      */
     axis: 'block' | 'inline' | 'none';
+    /**
+     * Whether the plot is drawn left to right on every page, as Scatter Chart's
+     * plane is, so the arrows follow the drawing rather than the page's
+     * direction. Off, the arrow pointing along the line of text moves on.
+     */
+    drawnLtr: boolean;
     /**
      * Hears the reading: `change` each time it moves or is put away, and
      * `press` when it is clicked, tapped, or pressed with Enter or Space.

package/dist/controllers/chart-pointer-controller/chart-pointer-controller.js

@@ -10,6 +10,7 @@ export class ChartPointerController {
         this.keyed = false;
         this.series = -1;
         this.axis = 'inline';
+        this.drawnLtr = false;
         this.by = 'mouse';
         this.said = { at: -1, series: -1 };
         this.tapped = false;
@@ -87,7 +88,7 @@ export class ChartPointerController {
                 count,
                 grid: this.grid,
                 page: this.page(count),
-                rtl: isRtl(this.host),
+                rtl: !this.drawnLtr && isRtl(this.host),
                 series: this.columns().length,
             });
             if (next === undefined) {

package/dist/controllers/host-click-controller/host-click-controller.d.ts

@@ -10,10 +10,24 @@ import type { ReactiveController, ReactiveControllerHost } from 'lit';
  * under `click()` while a pointer press on it turned it on. A native checkbox
  * toggles on both, and code written against one expects it.
  *
- * **Only a click that starts at the host**, whose path begins there. A press
- * inside the shadow root reaches the host too, retargeted, and has already
- * done its work where it landed; the click this sends to the control reaches
- * the host the same way, and goes no further.
+ * **One click reaches the page for each press, as from a native control.**
+ * The control's click used to be sent composed, so it crossed the shadow root
+ * and reached the host after the host's own: a listener on the host heard two
+ * clicks for one `click()` or one press on a `<label for>`, where one on a
+ * native checkbox hears one. So there are two paths, and each leaves exactly
+ * one click outside the shadow root:
+ *
+ * - **The host's `click()`** calls {@link click}, which clicks the control
+ *   instead. That click is the only one: it reaches the host retargeted, as a
+ *   click inside a native control does, after the control has moved, so a
+ *   listener reads the new state and `preventDefault()` puts the old one back.
+ * - **A click that starts at the host** some other way, from a `<label for>`,
+ *   a press on the host's own box, or `dispatchEvent`, is the click the page
+ *   hears. The control is sent a copy that is not composed, so it moves the
+ *   control and stops at the shadow root. The copy goes in the capture phase,
+ *   so the host's own listeners already read the new state, as a native
+ *   checkbox's do. A press inside the shadow root has already landed on its
+ *   control and is left alone.
  */
 export declare class HostClickController implements ReactiveController {
     private readonly host;
@@ -21,5 +35,11 @@ export declare class HostClickController implements ReactiveController {
     constructor(host: ReactiveControllerHost & HTMLElement, target: () => HTMLElement | null | undefined);
     hostConnected(): void;
     hostDisconnected(): void;
+    /**
+     * Clicks the control, for the host's own `click()`. Answers `false` when
+     * there is no control yet, before the first render, so the host can fall
+     * back to its own click.
+     */
+    click(): boolean;
     private readonly forward;
 }

package/dist/controllers/host-click-controller/host-click-controller.js

@@ -3,17 +3,44 @@ export class HostClickController {
         this.host = host;
         this.target = target;
         this.forward = (event) => {
-            if (event.composedPath()[0] !== this.host) {
+            if (event.composedPath()[0] !== this.host || event.defaultPrevented) {
                 return;
             }
-            this.target()?.click();
+            const control = this.target();
+            if (!control || control.matches(':disabled')) {
+                return;
+            }
+            control.dispatchEvent(new MouseEvent('click', {
+                altKey: event.altKey,
+                bubbles: true,
+                button: event.button,
+                buttons: event.buttons,
+                cancelable: true,
+                clientX: event.clientX,
+                clientY: event.clientY,
+                composed: false,
+                ctrlKey: event.ctrlKey,
+                detail: event.detail,
+                metaKey: event.metaKey,
+                screenX: event.screenX,
+                screenY: event.screenY,
+                shiftKey: event.shiftKey,
+            }));
         };
         host.addController(this);
     }
     hostConnected() {
-        this.host.addEventListener('click', this.forward);
+        this.host.addEventListener('click', this.forward, { capture: true });
     }
     hostDisconnected() {
-        this.host.removeEventListener('click', this.forward);
+        this.host.removeEventListener('click', this.forward, { capture: true });
+    }
+    click() {
+        const control = this.target();
+        if (!control) {
+            return false;
+        }
+        control.click();
+        return true;
     }
 }

package/dist/index.d.ts

@@ -1,3 +1,4 @@
+export { auditChildren } from '@fusion.dev/helix-core/audits/children';
 export { auditName } from '@fusion.dev/helix-core/audits/name';
 export { auditNesting } from '@fusion.dev/helix-core/audits/nesting';
 export { auditServed } from '@fusion.dev/helix-core/audits/served';

package/dist/index.js

@@ -1,3 +1,4 @@
+export { auditChildren } from '@fusion.dev/helix-core/audits/children';
 export { auditName } from '@fusion.dev/helix-core/audits/name';
 export { auditNesting } from '@fusion.dev/helix-core/audits/nesting';
 export { auditServed } from '@fusion.dev/helix-core/audits/served';

package/dist/utils/audit-children/audit-children-off.d.ts

@@ -0,0 +1,10 @@
+import type { auditChildren as audit } from './audit-children.js';
+/**
+ * The unregistered-child warning in a production build: nothing.
+ *
+ * `#audit-children` resolves here unless the bundler or runtime asks for the
+ * `development` condition, so a production build carries none of the audit's
+ * code. The signature is the audit's, so a call site reads the same in both
+ * builds. See `audit-children.ts`.
+ */
+export declare const auditChildren: typeof audit;

package/dist/utils/audit-children/audit-children-off.js

@@ -0,0 +1 @@
+export const auditChildren = () => { };

package/dist/utils/audit-children/audit-children.d.ts

@@ -0,0 +1,28 @@
+/**
+ * Reports a child from another family that a container holds and nobody
+ * registered, naming the entry to import.
+ *
+ * **A container does not register another family's children.** An option
+ * belongs to the Option family, and a page writing one inside a Select imports
+ * `@fusion.dev/helix/option` itself, as it imports the Select. Left out, the
+ * option is an unknown element: the list reads no choices and nothing throws.
+ * This says so, once for each container tag and child tag on the page, with
+ * the line to add.
+ *
+ * Asked once the page has loaded and painted, so an entry a later module
+ * imports is not taken for a missing one.
+ *
+ * **In a development build only**, as `missing-name` is: every container would
+ * pay for a sentence meant for the person writing the page. Core imports it as
+ * `#audit-children`, which `package.json` resolves here under the
+ * `development` condition and to `audit-children-off.ts`, which does nothing,
+ * everywhere else.
+ *
+ * `part` is the child's tag after the tier, such as `option`, and `entry` its
+ * family's entry, such as `option`; the host's `tier` gives the rest of both.
+ * `consequence` says what the missing child costs, after "so". `drawn` is for
+ * children the host draws from data, such as the options of Listbox's `items`:
+ * the page wrote none, so an undefined tag is reported without one in the
+ * host's light DOM.
+ */
+export declare const auditChildren: (host: HTMLElement, part: string, entry: string, consequence: string, drawn?: boolean) => void;

package/dist/utils/audit-children/audit-children.js

@@ -0,0 +1,37 @@
+import { reportRuntime } from '../report-runtime/report-runtime.js';
+const reported = new Set();
+export const auditChildren = (host, part, entry, consequence, drawn = false) => {
+    if (typeof window === 'undefined') {
+        return;
+    }
+    const tier = host.constructor.tier ?? 'helix-headless-';
+    const tag = `${tier}${part}`;
+    const key = `${host.localName}>${tag}`;
+    if (reported.has(key)) {
+        return;
+    }
+    const check = () => {
+        if (reported.has(key) ||
+            !host.isConnected ||
+            customElements.get(tag) !== undefined ||
+            (!drawn && host.querySelector(tag) === null)) {
+            return;
+        }
+        reported.add(key);
+        const pkg = tier === 'helix-' ? '@fusion.dev/helix' : '@fusion.dev/helix-headless';
+        reportRuntime({
+            kind: 'unregistered-part',
+            message: `<${tag}> in <${host.localName}> is not registered, so ${consequence}. Import '${pkg}/${entry}'.`,
+            tag: host.localName,
+        });
+    };
+    const frame = () => {
+        requestAnimationFrame(check);
+    };
+    if (document.readyState === 'complete') {
+        frame();
+    }
+    else {
+        window.addEventListener('load', frame, { once: true });
+    }
+};

package/dist/version.d.ts

@@ -6,4 +6,4 @@
  * release, so the two never disagree; a spec beside this holds them to it.
  * Nothing else in the running code knows which release it is.
  */
-export declare const VERSION = "0.179.0";
+export declare const VERSION = "0.180.1";

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = '0.179.0';
+export const VERSION = '0.180.1';

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@fusion.dev/helix-core",
   "description": "Helix element primitives: base classes, mixins, decorators, utilities.",
-  "version": "0.179.0",
+  "version": "0.180.1",
   "license": "MIT",
   "author": "Ross Sheppard <ross.sheppard@fusion.dev>",
   "repository": {
@@ -17,6 +17,12 @@
       "types": "./dist/index.d.ts",
       "default": "./dist/index.js"
     },
+    "./audits/children": {
+      "types": "./dist/utils/audit-children/audit-children.d.ts",
+      "development": "./dist/utils/audit-children/audit-children.js",
+      "helix-development": "./dist/utils/audit-children/audit-children.js",
+      "default": "./dist/utils/audit-children/audit-children-off.js"
+    },
     "./audits/name": {
       "types": "./dist/utils/audit-name/audit-name.d.ts",
       "development": "./dist/utils/audit-name/audit-name.js",
@@ -68,16 +74,21 @@
   "bugs": "https://github.com/project-fusion/fusion/issues",
   "dependencies": {
     "@floating-ui/dom": "^1.8.0",
-    "@fusion.dev/helix-icons": "0.179.0"
+    "@fusion.dev/helix-icons": "0.180.1"
   },
   "devDependencies": {
-    "@fusion.dev/helix-cem": "0.179.0",
-    "@fusion.dev/helix-testing": "0.179.0",
+    "@fusion.dev/helix-cem": "0.180.1",
+    "@fusion.dev/helix-testing": "0.180.1",
     "@lit-labs/ssr": "^4.1.0",
     "lit": "^3.2.1"
   },
   "homepage": "https://fusion.dev",
   "imports": {
+    "#audit-children": {
+      "development": "./src/utils/audit-children/audit-children.ts",
+      "helix-development": "./src/utils/audit-children/audit-children.ts",
+      "default": "./src/utils/audit-children/audit-children-off.ts"
+    },
     "#audit-name": {
       "development": "./src/utils/audit-name/audit-name.ts",
       "helix-development": "./src/utils/audit-name/audit-name.ts",