@wildwinter/expr-editor 0.15.0 → 0.15.1

package/dist/index.js

@@ -383,8 +383,14 @@ function openPopover(anchor, render, opts = {}) {
   container.append(pop);
   const a = anchor.getBoundingClientRect();
   const ph = pop.getBoundingClientRect();
-  let top = a.bottom + 4;
-  if (top + ph.height > window.innerHeight - 8 && a.top - ph.height - 4 > 8) top = a.top - ph.height - 4;
+  const GAP = 4;
+  const EDGE = 8;
+  const below = window.innerHeight - EDGE - (a.bottom + GAP);
+  const above = a.top - GAP - EDGE;
+  const flip = ph.height > below && above > below;
+  const room = flip ? above : below;
+  const top = flip ? a.top - Math.min(ph.height, above) - GAP : a.bottom + GAP;
+  if (ph.height > room) pop.style.maxHeight = `${Math.max(96, Math.round(room))}px`;
   let left = a.left;
   if (left + ph.width > window.innerWidth - 8) left = Math.max(8, window.innerWidth - 8 - ph.width);
   if (container === document.body) {
@@ -1385,6 +1391,7 @@ function valueWizard(opts) {
   const isStage = opts.expectedType === "quality";
   const flagsReady = opts.expectedType === "flags" && opts.expectedRef !== void 0;
   let kind = opts.expectedType === "number" ? "number" : opts.expectedType === "string" ? "text" : opts.expectedType === "boolean" ? "bool" : (opts.expectedType === "enum" || opts.expectedType === "quality") && choices?.length ? "enum" : flagsReady ? "flags" : "menu";
+  let flagSign = "+";
   const draw = () => {
     body.replaceChildren();
     const other = () => button("exed-vwiz-other", "\u21A9 a different kind", () => {
@@ -1421,12 +1428,23 @@ function valueWizard(opts) {
       case "flags": {
         const flags = opts.expectedFlags ?? [];
         if (!flags.length) body.append(el("div", "exed-hint", ["This property declares no flag values yet."]));
-        for (const sign of ["+", "-"]) {
-          if (!flags.length) break;
-          body.append(el("div", "exed-vwiz-note", [sign === "+" ? "Set a flag:" : "Clear a flag:"]));
-          const row = el("div", "exed-field-row");
-          for (const f of flags) row.append(optBtn(`${sign} ${f}`, () => opts.onCommit(flagChangeSrc(opts.expectedRef, sign, f))));
-          body.append(row);
+        else {
+          const signRow = el("div", "exed-field-row");
+          signRow.append(
+            button(`exed-opt${flagSign === "+" ? " sel" : ""}`, "\uFF0B set", () => {
+              flagSign = "+";
+              draw();
+            }),
+            button(`exed-opt${flagSign === "-" ? " sel" : ""}`, "\uFF0D clear", () => {
+              flagSign = "-";
+              draw();
+            })
+          );
+          body.append(signRow);
+          body.append(el("div", "exed-vwiz-note", [flagSign === "+" ? "Set a flag:" : "Clear a flag:"]));
+          const list = el("div", "exed-picker-list");
+          for (const f of flags) list.append(optBtn(`${flagSign} ${f}`, () => opts.onCommit(flagChangeSrc(opts.expectedRef, flagSign, f))));
+          body.append(list);
         }
         body.append(button("exed-vwiz-other", "\u21A9 a different kind (replaces every flag)", () => {
           kind = "menu";