u2ui 1.5.17 → 1.5.19

package/el/carousel/carousel.js

@@ -103,6 +103,7 @@ contain: layout style paint;  /* Neu ok? */
 			}
 			:host([mode=fade]) .body {
 				display:flex;
+				min-width: 0;
 			}
 			:host([mode=fade]) .body::slotted(*) {
 				transition:opacity var(--u2-carousel-animation-speed, .7s) ease-in-out;

package/el/fields/fields.js

@@ -1,3 +1,5 @@
+// todo: shadowrootreferencetarget
+
 import U2Element from '../../u2/Element.js';
 export default class U2Fields extends U2Element {
     constructor() {

package/js/SelectorObserver/SelectorObserver.js

@@ -12,19 +12,18 @@ function forRoot(root, method, el) {
     for (const observer of observers) observer.root === root && observer[method](el);
 }
 function checkMutations(mutations, root) {
-    for (const mutation of mutations) {
-        if (mutation.type==='childList') {
-            for (const target of mutation.addedNodes) {
-                target.nodeType === 1 && forRoot(root, '_addTree', target);
-            }
-            for (const target of mutation.removedNodes) {
-                target.nodeType === 1 && forRoot(root, '_removeTree', target);
-            }
-        }
-        if (mutation.type==='attributes') {
-            forRoot(root, '_treeModified', mutation.target);
-        }
+    const added = new Set(), removed = new Set(), modified = new Set();
+    for (const m of mutations) {
+        if (m.type === 'attributes') { modified.add(m.target); continue; }
+        for (const n of m.removedNodes) n.nodeType === 1 && removed.add(n);
+        for (const n of m.addedNodes) n.nodeType === 1 && added.add(n);
     }
+    // a node inside another node of the same batch is covered by that node's subtree
+    const top = (n, set) => { for (let p = n.parentNode; p; p = p.parentNode) if (set.has(p)) return false; return true; };
+    for (const n of removed) top(n, removed) && forRoot(root, '_removeTree', n);
+    // judged by the final state: a node that left again, or was only touched while detached, is not added
+    for (const n of added) top(n, added) && root.contains(n) && forRoot(root, '_addTree', n);
+    for (const n of modified) root.contains(n) && forRoot(root, '_treeModified', n);
 }
 
 
@@ -140,16 +139,15 @@ export class SelectorObserver {
      */
      _addTree(target) {
         target.matches(this.selector) && this._add(target);
-        if (document.readyState === 'complete') { // Before domready MutationObserver reports every node, ok?
-            for (const el of target.querySelectorAll(this.selector)) this._add(el);
-        }
+        // a scripted insert is reported as its root only, even while parsing; nodes found twice are deduped by targets
+        for (const el of target.querySelectorAll(this.selector)) this._add(el);
     }
 
     /**
      * @private
      */
      _removeTree(target) {
-        //if (!this._off) return; // performance
+        if (!this.targets.size) return;
         this._remove(target);
         for (const el of target.querySelectorAll('*')) this._remove(el);
     }

package/js/contextmenu/contextmenu.js

@@ -1,4 +1,3 @@
-
 class Menu {
     constructor(menuItem) {
         this.menuItem = menuItem;
@@ -323,6 +322,9 @@ document.documentElement.addEventListener('contextmenu', e=>{
     e.preventDefault();
     
     rootEl.showPopover();
+    // pointerup still pending (not on Windows): capture it, so it does not light-dismiss the popover
+    if (e.buttons) rootEl.setPointerCapture(e.pointerId);
+
     rootEl.querySelector('button').focus();
 
     let top  = e.clientY + 2;

package/js/rte/src/client/README.md

@@ -402,8 +402,8 @@ its replacement, as a promise or directly. `surface` comes along rather than a
 digest of it: `surface.config` says what the field allows — with `null` where
 nothing is restricted — because an answer using anything else is cleaned away on
 the way back in, and `surface.element` is where an application reads whatever
-else its own fields carry; a rejection is shown in the answer
-pane and leaves the field untouched. `prompts` fills a datalist beside the
+else its own fields carry; a rejection is shown as an alert
+(`js/dialog`) and leaves the panes and the field untouched. `prompts` fills a datalist beside the
 input. Without `diff` the dialog shows two panes — comparing two HTML strings is
 a library question, not an editor one, so none is pulled in. Answers that arrive
 after the next prompt, or after the dialog closed, are dropped.

package/js/rte/src/client/ai.js

@@ -148,7 +148,11 @@ async function ask(state) {
         parts.apply.disabled = false;
         await update(state, pending, run);
     } catch (error) {
-        if (state.pending === pending && pending.run === run) parts.answer.textContent = String(error?.message ?? error);
+        // A failure is no answer: it comes as an alert, and the panes keep what they had.
+        if (state.pending === pending && pending.run === run) {
+            const {alert} = await import('../../../dialog/dialog.js');
+            alert(String(error?.message ?? error), {root: state.dialog.parentNode});
+        }
     } finally {
         if (state.pending === pending && pending.run === run) parts.busy.disabled = false;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "u2ui",
-  "version": "1.5.17",
+  "version": "1.5.19",
   "description": "HTML Advanced: More Capabilities, Same Feel, Same Simplicity",
   "keywords": ["web-components","pattern-library","component-library","ui-toolkit","design-system"],
   "author": "Tobias Buschor",