mnfst 0.5.168 → 0.5.169

package/lib/manifest.combobox.js

@@ -497,13 +497,31 @@ function initializeComboboxPlugin() {
         let suppressOpen = false;
         function refocus() { suppressOpen = true; el.focus(); setTimeout(() => { suppressOpen = false; }, 0); }
 
+        // The model is written ONLY on explicit user selection; commits also fire a DOM
+        // change event so apps can save on events instead of watching the model.
+        // `dirty` = the user actually edited the text since focus — gates Enter's
+        // free-text commit so a stray Enter on an untouched field can't re-commit the
+        // display label (or, with the menu open, an auto-highlighted option) as a value.
+        let dirty = false;
+        let seeding = false;   // init-time seeding is not a user commit — no change event
+        const fireChange = () => { if (!seeding) el.dispatchEvent(new Event('change', { bubbles: true })); };
+
         function addValue(value, label) {
-            if (!multiple) { selected = [{ value, label }]; render(); syncOut(); announce(label + ' selected'); return; }
+            if (!multiple) {
+                const same = selected[0] && String(selected[0].value) === String(value);
+                selected = [{ value, label }];
+                render();
+                dirty = false;
+                if (!same) { syncOut(); fireChange(); announce(label + ' selected'); }
+                return;
+            }
             if (isSelected(value)) return;
             if (selected.length >= max) { announce('Maximum of ' + max + ' reached'); return; }
             selected.push({ value, label });
             render();
+            dirty = false;
             syncOut();
+            fireChange();
             announce(label + ' added');
         }
         function removeValue(value) {
@@ -513,6 +531,7 @@ function initializeComboboxPlugin() {
             const [g] = selected.splice(i, 1);
             render();
             syncOut();
+            fireChange();
             announce(g.label + ' removed');
             if (menu && !isAsync) filter();
             refocus();
@@ -634,7 +653,7 @@ function initializeComboboxPlugin() {
                 if (e.key === 'ArrowDown') { e.preventDefault(); moveActive(1); }
                 else if (e.key === 'ArrowUp') { e.preventDefault(); moveActive(-1); }
                 else if (e.key === 'Enter' || e.key === ' ') {
-                    if (activeIndex >= 0 || activeIndex === -2) { e.preventDefault(); selectOption(liAt(activeIndex)); }
+                    if (!e.repeat && menu.hasAttribute('data-kbd') && (activeIndex >= 0 || activeIndex === -2)) { e.preventDefault(); selectOption(liAt(activeIndex)); }
                 }
                 else if (e.key === 'Escape') { if (open) { e.preventDefault(); closeMenu(); } }
             });
@@ -643,10 +662,15 @@ function initializeComboboxPlugin() {
                 if (e.key === 'ArrowDown' && menu) { e.preventDefault(); openMenu(); moveActive(1); }
                 else if (e.key === 'ArrowUp' && menu) { e.preventDefault(); moveActive(-1); }
                 else if (e.key === 'Enter') {
-                    if (menu && menu.matches(':popover-open') && (activeIndex >= 0 || activeIndex === -2)) {
+                    // Commit only on deliberate action: an option needs a keyboard-driven
+                    // highlight (data-kbd — arrows or typing), free text needs an actual
+                    // edit since focus (dirty). A stray Enter on an untouched field — key
+                    // repeat, form submit, saving the record — must never write the model.
+                    if (e.repeat) return;
+                    if (menu && menu.matches(':popover-open') && menu.hasAttribute('data-kbd') && (activeIndex >= 0 || activeIndex === -2)) {
                         e.preventDefault();
                         selectOption(liAt(activeIndex));
-                    } else if (!strict || !menu) {
+                    } else if ((!strict || !menu) && dirty) {
                         if (commitText(el.value)) e.preventDefault();
                     }
                 }
@@ -658,19 +682,27 @@ function initializeComboboxPlugin() {
 
             // Separators (and paste) commit from here, the moment one lands in the value.
             el.addEventListener('input', () => {
+                dirty = true;
                 if (menu) menu.setAttribute('data-kbd', '');   // typing drives the highlight onto the first match
                 extractTokens();
                 if (isAsync) asyncRefresh(); else { openMenu(); filter(); }
             });
 
-            el.addEventListener('focus', () => { if (!suppressOpen) openForEntry(true); });
+            el.addEventListener('focus', () => { dirty = false; if (!suppressOpen) openForEntry(true); });
             // Reopen on click even when the field is already focused (focus won't re-fire).
             el.addEventListener('mousedown', () => { if (menu && !menu.matches(':popover-open') && !suppressOpen) openForEntry(false); });
         }
 
-        // Close when focus leaves the field entirely (Tab away).
+        // Close when focus leaves the field entirely (Tab away). A committed single
+        // field also snaps its display back to the selection's label — abandoned typing
+        // never commits, and the model is left untouched.
         el.addEventListener('blur', () => setTimeout(() => {
-            if (menu && !wrap.contains(document.activeElement) && !menu.contains(document.activeElement)) closeMenu();
+            if (wrap.contains(document.activeElement) || (menu && menu.contains(document.activeElement))) return;
+            closeMenu();
+            if (!multiple && !chips && !editorNone && selected[0] && el.value !== selected[0].label) {
+                el.value = selected[0].label;
+                dirty = false;
+            }
         }, 0));
 
         // Click empty shell → focus the trigger
@@ -753,7 +785,9 @@ function initializeComboboxPlugin() {
         if (!modelExpr && !adopt && !editorNone && chips && el.value) {
             const seeds = el.value.split(/[,\n;]+/).map(s => s.trim()).filter(Boolean);
             el.value = '';
+            seeding = true;
             seeds.forEach(s => commitText(s));
+            seeding = false;
         }
         // In chips/multiple the editor is a typing buffer. Clear value AND remove the
         // attribute: mnfst-render serializes the attribute, so leaving it bakes seed text

package/lib/manifest.integrity.json

@@ -7,7 +7,7 @@
   "manifest.code.js": "sha384-G3MNfvQRWWY+gm4SGGrSsgQbDeil9MP+ON2DTk6Rcq9Nsex8woHMDt5EJHrFH0Nj",
   "manifest.color.js": "sha384-6Rv3LxyTcZNjrhtayQfqRdCx0uSZ4BiEbgEI98I62eTvp8Aw7LBIoNJ0Je1oktwL",
   "manifest.colorpicker.js": "sha384-N5ezEwgKrDodV4YdURxhdyZKmg1n/QhqvmAC4SEG03mw1pEAZAA8Gphp+NLAOxts",
-  "manifest.combobox.js": "sha384-tK8cVTQy3v4RK8pDJKFU+UW7zdsS7ysMqExOsCoWpFBXD0tMfjLNrkMRCQd4+tsW",
+  "manifest.combobox.js": "sha384-ZPOkx4hKGlcrOqeb3MmWLyBfjZXlnkP47Yc9VLE7yHds6dXRqMeB5DPFeJl3NZS/",
   "manifest.components.js": "sha384-eZDib+RFdwd9mOUdi6o53eNS0LBlEEkZPDegxtkAcE+GrGIFxsuB4O0ymfixGNfM",
   "manifest.data.js": "sha384-3lLvGnqcQr4Ium80MdaQTr8RytjVbs5rnxd0q41BF3ppcoQccMFAcvgFSyZP/0sl",
   "manifest.datepicker.js": "sha384-e6rJI0BxuE+AyI96/u6XoeOstsxLuQTO6+wYSr6rm2aKhOP2APM2xJqTz//0v4Pw",

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "mnfst",
-    "version": "0.5.168",
+    "version": "0.5.169",
     "private": false,
     "workspaces": [
         "templates/starter",