mnfst 0.5.195 → 0.5.197

package/lib/manifest.chat.js

@@ -106,11 +106,24 @@
             for (const k of Object.keys(incoming)) if (k !== 'body' && k !== '_seq') target[k] = incoming[k];
         }
 
+        // own echo beating the ack: claim the pending row by meta.clientId
+        function claimPending(incoming) {
+            const cid = incoming.meta && incoming.meta.clientId;
+            if (cid == null || !state.me || !incoming.author || incoming.author.id !== state.me.id) return null;
+            return _msgs.find(m => m._optimistic && m._clientId === cid) || null;
+        }
+
         function upsert(raw) {
             const incoming = normalize(raw, ++seq);
             const prev = incoming.id != null ? _byId.get(incoming.id) : null;
+            const pending = prev ? null : claimPending(incoming);
             if (prev) {
                 merge(prev, incoming);
+            } else if (pending) {
+                _byId.delete(pending.id);
+                merge(pending, incoming);
+                pending._optimistic = false;
+                if (pending.id != null) _byId.set(pending.id, pending);
             } else {
                 _msgs.push(incoming);
                 if (incoming.id != null) _byId.set(incoming.id, incoming);
@@ -176,11 +189,11 @@
         async function send(draft) {
             const tmp = 'tmp_' + (++seq);
             const body = draft && draft.body ? draft.body : { text: draft && draft.text != null ? draft.text : '' };
-            const local = normalize({ id: tmp, conversationId, author: state.me, body, replyTo: draft && draft.replyTo, status: 'pending', ts: null, _optimistic: true }, ++seq);
+            const local = normalize({ id: tmp, conversationId, author: state.me, body, replyTo: draft && draft.replyTo, status: 'pending', ts: null, _optimistic: true, _clientId: tmp }, ++seq);
             _msgs.push(local); _byId.set(tmp, local); commit();
             if (!adapter.send) { local.status = 'failed'; local.statusReason = 'unsupported'; commit(); return; }
             try {
-                const ack = await adapter.send(conversationId, Object.assign({}, draft, { body }));
+                const ack = await adapter.send(conversationId, Object.assign({}, draft, { body, clientId: tmp }));
                 _byId.delete(tmp);
                 // 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.

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();
                                 }