@optionfactory/fml 9.0.0-rc23 → 9.0.0-rc24

package/dist/ful.mjs

@@ -693,6 +693,11 @@ class Bindings {
      * `rest` being the remainder of the path (`''` on an exact match) that a
      * composite can use to route the problem to an inner control.
      *
+     * A control counts only when it can show a problem, through a
+     * `setCustomValidity` of its own: a form-associated element that has none is
+     * skipped, so its problems fall to a shorter prefix or to the banner instead
+     * of being dropped.
+     *
      * Every other problem, and a field problem naming no control, is shown in
      * each `ful-errors` banner of the form: the banner gets `role="alert"`, is
      * revealed and holds the reasons one per line. With no such problem the
@@ -731,7 +736,7 @@ class Bindings {
             const parts = name.split('.');
             for (let i = parts.length; i !== 0; --i) {
                 const prefix = parts.slice(0, i).join('.');
-                const targets = named.get(prefix) ?? [];
+                const targets = (named.get(prefix) ?? []).filter((el) => typeof el.setCustomValidity === 'function');
                 if (targets.length === 0) {
                     continue;
                 }
@@ -1189,6 +1194,14 @@ class Field extends ParsedElement {
      *   accessibility tree.
      * - `also` are further controls that mirror `disabled` and `readOnly`
      *   beside the claim target.
+     *
+     * The pieces are elements the field rendered itself, native controls or
+     * wrappers of its own. A ful field is not one: it has no `readOnly`, its
+     * host's role takes no aria, and a label's `for` names its host rather than
+     * its control. A composite built from ful fields answers the inner field as
+     * `control`, which forwards `focus()`, lets it render its own label with
+     * `label` and `announces` null, and forwards the claims in its setters,
+     * calling super first.
      * @param {{ slots: Record<string, DocumentFragment> | undefined }} conf
      * @returns {any} the pieces, or a promise of them
      * @throws {Error} when the subclass does not implement it
@@ -4576,7 +4589,7 @@ class RadioGroup extends Field {
             return;
         }
         /** @type {HTMLInputElement|null} */
-        const el = this.querySelector(`input[type=radio][value=${CSS.escape(String(value))}]`);
+        const el = this.querySelector(`input[type=radio][value="${CSS.escape(String(value))}"]`);
         if (el === null) {
             clear();
             return;
@@ -6850,11 +6863,12 @@ class DialogSections {
      * `show`, and fills the content with what the callback resolves to. A
      * rejection is shown, as the problems' reasons, in the error section, and
      * the content is hidden. When a newer `update()` starts before the callback
-     * settles, this call paints nothing: it resolves with the section regardless,
-     * and still rejects with the callback's error.
+     * settles, this call paints nothing and resolves `null`, the section being
+     * the newer call's; it still rejects with the callback's error.
      * @param {() => Node|string|Promise<Node|string>} cb producing the content
      * @param {() => void} show opens the dialog when it is closed
-     * @returns {Promise<HTMLElement>} the content section
+     * @returns {Promise<HTMLElement|null>} the content section, or `null` for a
+     * call a newer one overtook
      * @throws {any} what the callback threw or rejected with
      */
     async update(cb, show) {
@@ -6867,7 +6881,7 @@ class DialogSections {
         try {
             const delivered = await cb();
             if (claim.stale) {
-                return this.#section;
+                return null;
             }
             this.#section.replaceChildren(delivered);
             this.#loading?.setAttribute('hidden', '');
@@ -7113,11 +7127,12 @@ class Dialog extends ParsedElement {
      *
      * No `section:requested` is dispatched, and no `ask()` waiter is added:
      * listen for the `close` event to learn how the dialog ended. When a newer
-     * `update()` starts before the callback settles, this call paints nothing:
-     * it resolves with the body regardless, and still rejects with the
-     * callback's error.
+     * `update()` starts before the callback settles, this call paints nothing
+     * and resolves `null`, the body being the newer call's; it still rejects
+     * with the callback's error.
      * @param {() => Node|string|Promise<Node|string>} cb producing the body content
-     * @returns {Promise<Element>} the body section, in the document
+     * @returns {Promise<Element|null>} the body section, in the document, or
+     * `null` for a call a newer one overtook
      * @throws {any} what the callback threw or rejected with
      */
     async update(cb) {
@@ -7407,12 +7422,13 @@ class Drawer extends ParsedElement {
      * which has `role="alert"`, and the content is hidden.
      *
      * No `section:requested` is dispatched. When a newer `update()` starts
-     * before the callback settles, this call paints nothing: it resolves with
-     * the content section regardless, and still rejects with the callback's
-     * error.
+     * before the callback settles, this call paints nothing and resolves
+     * `null`, the content section being the newer call's; it still rejects with
+     * the callback's error.
      * @param {string} header the title, set as text
      * @param {() => Node|string|Promise<Node|string>} cb producing the content
-     * @returns {Promise<Element>} the content section, in the document
+     * @returns {Promise<Element|null>} the content section, in the document, or
+     * `null` for a call a newer one overtook
      * @throws {any} what the callback threw or rejected with
      */
     async update(header, cb) {