@wildwinter/expr-editor 0.6.0 → 0.7.0

package/dist/index.cjs

@@ -458,6 +458,14 @@ function renderNode(node, path, ctx, parentOp, side) {
     }
   }
 }
+function signToggle(current, onPick) {
+  const row = el("div", "exed-sign");
+  ["+", "-"].forEach((s) => {
+    const cls = `exed-signbtn${s === current ? s === "+" ? " sel-pos" : " sel-neg" : ""}`;
+    row.append(button(cls, s === "+" ? "+ set" : "\u2212 unset", () => onPick(s)));
+  });
+  return row;
+}
 var deleteOrUnwrapNot = (ctx, path, node) => (
   // clicking the NOT pill strips it (keeps the operand)
   setNodeAt(ctx.getAst(), path, node.operand)
@@ -547,14 +555,10 @@ function flagEditor(ctx, path, node, anchor) {
   ctx.openPopover(anchor, (close) => {
     const wrap = el("div", "exed-menu");
     wrap.append(el("div", "exed-menu-head", ["Flag set?"]));
-    const signRow = el("div", "exed-field-row");
-    for (const s of ["+", "-"]) {
-      signRow.append(button(`exed-opt${s === node.sign ? " sel" : ""}`, s === "+" ? "+ set" : "\u2212 unset", () => {
-        replace(ctx, path, flagDelta(s, node.name));
-        close();
-      }));
-    }
-    wrap.append(signRow);
+    wrap.append(signToggle(node.sign, (s) => {
+      replace(ctx, path, flagDelta(s, node.name));
+      close();
+    }));
     const names = (entry?.enumValues ?? []).filter((n) => !used.includes(n));
     if (names.length) {
       wrap.append(el("div", "exed-menu-head", ["Flag"]));
@@ -1164,21 +1168,15 @@ function mountExpressionEditor(host, opts) {
         let sign = "+";
         const wrap = el("div", "exed-menu");
         wrap.append(el("div", "exed-menu-head", ["Add flag"]));
-        const signRow = el("div", "exed-field-row");
+        const signHost = el("div");
         const drawSign = () => {
-          signRow.replaceChildren(
-            button(`exed-opt${sign === "+" ? " sel" : ""}`, "+ set", () => {
-              sign = "+";
-              drawSign();
-            }),
-            button(`exed-opt${sign === "-" ? " sel" : ""}`, "\u2212 unset", () => {
-              sign = "-";
-              drawSign();
-            })
-          );
+          signHost.replaceChildren(signToggle(sign, (s) => {
+            sign = s;
+            drawSign();
+          }));
         };
         drawSign();
-        wrap.append(signRow);
+        wrap.append(signHost);
         const commit = (name) => {
           const delta = flagDelta(sign, name);
           emit(toSrc(call ? { ...call, args: [...call.args, delta] } : callNode("set_flags", [parseRef(opts.flagValue.target), delta])));