mnfst 0.5.194 → 0.5.196

package/lib/manifest.chat.js

@@ -100,13 +100,17 @@
         function commitParticipants() { state.participants = [..._participants.values()]; rev().n++; }
         function commitTyping() { state.typing = [..._typing.values()]; rev().n++; }
 
+        // server echo reconciles body/media/status onto the local copy
+        function merge(target, incoming) {
+            Object.assign(target.body, incoming.body);
+            for (const k of Object.keys(incoming)) if (k !== 'body' && k !== '_seq') target[k] = incoming[k];
+        }
+
         function upsert(raw) {
             const incoming = normalize(raw, ++seq);
             const prev = incoming.id != null ? _byId.get(incoming.id) : null;
             if (prev) {
-                // merge — server echo reconciles body/media/status onto the local copy
-                Object.assign(prev.body, incoming.body);
-                for (const k of Object.keys(incoming)) if (k !== 'body' && k !== '_seq') prev[k] = incoming[k];
+                merge(prev, incoming);
             } else {
                 _msgs.push(incoming);
                 if (incoming.id != null) _byId.set(incoming.id, incoming);
@@ -178,7 +182,16 @@
             try {
                 const ack = await adapter.send(conversationId, Object.assign({}, draft, { body }));
                 _byId.delete(tmp);
-                local.id = ack.id; local.ts = ack.ts; local.status = 'sent'; local._optimistic = false;
+                // A bus that echoes our own send can beat the ack; that echo lands
+                // while this row is still tmp-keyed, so it appended as its own entry.
+                const echo = ack.id != null ? _byId.get(ack.id) : null;
+                if (echo && echo !== local) {
+                    merge(local, echo);
+                    const at = _msgs.indexOf(echo); if (at > -1) _msgs.splice(at, 1);
+                }
+                local.id = ack.id; local._optimistic = false;
+                if (ack.ts != null) local.ts = ack.ts;
+                if (local.status === 'pending') local.status = 'sent';
                 if (ack.conversationId && ack.conversationId !== local.conversationId) {
                     const from = local.conversationId; local.conversationId = ack.conversationId;
                     if (!isAggregate) state.lastRehome = { from, to: ack.conversationId };   // single-chat: cockpit must retarget

package/lib/manifest.colorpicker.css

@@ -10,13 +10,18 @@
 
 @layer base {
 
+    /* Animates `scale`, not `transform`. With fill mode `both` a transform
+       animation keeps its final value forever, which silently overrides any
+       transform an author put on a popover — so a menu positioned with
+       translate(-50%) simply would not move. The standalone property leaves
+       transform free. */
     @keyframes mnfst-popover-in {
         from {
-            transform: scale(.9)
+            scale: .9
         }
 
         to {
-            transform: none
+            scale: 1
         }
     }
 

package/lib/manifest.colorpicker.js

@@ -2547,6 +2547,11 @@ function initializeColorpickerPlugin() {
 
                     const isDialog = target.tagName === 'DIALOG';
 
+                    // A generated panel is appended to <body>, so manifest.dialog.css can't
+                    // tell from DOM position that it belongs to a swatch inside a dialog
+                    // rather than to the page behind it, and fades it out. Say so here.
+                    if (el.closest('[popover]')) target.setAttribute('data-popover-nested', '');
+
                     // Non-dialog popover targets route through x-dropdown (anchored like
                     // a dropdown). <dialog> targets open imperatively on click instead
                     // (modal/centered, not trigger-anchored).
@@ -2568,7 +2573,9 @@ function initializeColorpickerPlugin() {
                             e.preventDefault();
                             try {
                                 if (target.hasAttribute('popover')) {
-                                    if (!target.matches(':popover-open')) target.showPopover();
+                                    // `source` names the swatch as the invoker, so opening the
+                                    // panel can't light-dismiss a dialog or menu it sits inside.
+                                    if (!target.matches(':popover-open')) target.showPopover({ source: el });
                                 } else {
                                     if (!target.open) target.showModal();
                                 }