@trops/dash-core 0.1.565 → 0.1.567

package/dist/index.js

@@ -59151,27 +59151,23 @@ var DashboardConfigModal = function DashboardConfigModal(_ref) {
     });
   }
 
-  // Bulk-apply: write this provider as the binding for every widget of
-  // this type that doesn't already have an explicit widget-level override.
+  // Bulk-apply: retarget every widget of this provider type to the
+  // newly-chosen provider name. The header text above the dropdown
+  // says "Apply one provider to every widget of this type, or adjust
+  // per-widget below" — bulk is the retarget-everything action;
+  // per-row overrides happen AFTER bulk if the user wants exceptions.
   //
-  // Override-detection consults the staged state first so a freshly
-  // staged unset is recognised as "no override" and gets bulked. A
-  // non-empty staged value is treated as an explicit pick and is
-  // preserved (bulk skips). Without this, the user could stage an
-  // unset and then click Bulk and the widget they just unset would
-  // be skipped because the *original* layer-1 value still lived on
-  // `b.layoutItem.selectedProviders`.
+  // Previous behavior skipped rows that already had an explicit pick,
+  // which meant the dropdown was a no-op when every row already had a
+  // value (the common case: every Slack widget on a dashboard inherits
+  // the same Slack provider). The user changing bulk from "Slack" to
+  // "Slack Dash Comms" would see zero rows update and Save stay
+  // disabled. The fill-blanks-only semantics fought the UI text and
+  // the user's mental model — every spreadsheet-style bulk operation
+  // overrides everything, with per-row tweaks happening afterward.
   function stageBulk(providerType, providerName) {
     var affected = effectiveBindings.filter(function (b) {
-      var _staged$b$widgetId2, _b$layoutItem;
-      if (b.providerType !== providerType) return false;
-      var stagedValue = (_staged$b$widgetId2 = staged[b.widgetId]) === null || _staged$b$widgetId2 === void 0 ? void 0 : _staged$b$widgetId2[providerType];
-      if (stagedValue !== undefined) {
-        // Empty/null staged value → user staged an unset → include
-        // in bulk. Non-empty → explicit pick, preserve it.
-        return !stagedValue;
-      }
-      return !((_b$layoutItem = b.layoutItem) !== null && _b$layoutItem !== void 0 && (_b$layoutItem = _b$layoutItem.selectedProviders) !== null && _b$layoutItem !== void 0 && _b$layoutItem[providerType]);
+      return b.providerType === providerType;
     });
     setStaged(function (prev) {
       var next = _objectSpread$a({}, prev);
@@ -60019,10 +60015,7 @@ function ListenersTab(_ref0) {
     className: "flex flex-col gap-3 h-full min-h-0",
     children: /*#__PURE__*/jsxRuntime.jsxs("div", {
       className: "flex flex-row gap-3 flex-1 min-h-0",
-      children: [/*#__PURE__*/jsxRuntime.jsxs(DashReact.Card2, {
-        padding: "",
-        shadow: "",
-        rounded: "rounded-lg",
+      children: [/*#__PURE__*/jsxRuntime.jsxs("div", {
         className: "w-56 flex-shrink-0 overflow-hidden flex flex-col",
         children: [/*#__PURE__*/jsxRuntime.jsx("div", {
           className: "px-3 py-2 text-xs font-semibold opacity-50 uppercase tracking-wider",
@@ -60067,10 +60060,7 @@ function ListenersTab(_ref0) {
           onAdd: onAdd,
           onRemove: onRemove
         })]
-      }) : /*#__PURE__*/jsxRuntime.jsx(DashReact.Card2, {
-        padding: "",
-        shadow: "",
-        rounded: "rounded-lg",
+      }) : /*#__PURE__*/jsxRuntime.jsx("div", {
         className: "flex-1 min-w-0 flex items-center justify-center text-sm opacity-60",
         children: "Pick a widget on the left to wire its handlers."
       })]
@@ -60105,45 +60095,47 @@ function HandlersColumn(_ref10) {
     }
     return m;
   }, [myWiring]);
-  return /*#__PURE__*/jsxRuntime.jsxs(DashReact.Card2, {
-    padding: "",
-    shadow: "",
-    rounded: "rounded-lg",
-    className: "w-56 flex-shrink-0 overflow-hidden flex flex-col",
-    children: [/*#__PURE__*/jsxRuntime.jsx("div", {
-      className: "px-3 py-2 text-xs font-semibold opacity-50 uppercase tracking-wider",
-      children: "Event Handlers"
-    }), /*#__PURE__*/jsxRuntime.jsx(DashReact.Divider, {}), /*#__PURE__*/jsxRuntime.jsx("div", {
-      className: "overflow-y-auto flex-1",
-      children: receiver.eventHandlers.length === 0 ? /*#__PURE__*/jsxRuntime.jsx("div", {
-        className: "text-xs opacity-50 text-center py-6 px-3",
-        children: "This widget declares no event handlers."
-      }) : receiver.eventHandlers.map(function (h) {
-        var isActive = h === selectedHandler;
-        var count = countsByHandler.get(h) || 0;
-        return /*#__PURE__*/jsxRuntime.jsxs("button", {
-          type: "button",
-          onClick: function onClick() {
-            return onSelectHandler(h);
-          },
-          className: "w-full text-left px-3 py-2 text-sm flex items-center justify-between gap-2 border-l-2 ".concat(isActive ? "bg-indigo-900/30 border-indigo-400" : "border-transparent hover:bg-white/5"),
-          children: [/*#__PURE__*/jsxRuntime.jsxs("span", {
-            className: "flex items-center gap-2 min-w-0",
-            children: [/*#__PURE__*/jsxRuntime.jsx(DashReact.FontAwesomeIcon, {
-              icon: "bolt",
-              className: "h-3 w-3 opacity-60 flex-shrink-0"
-            }), /*#__PURE__*/jsxRuntime.jsx("code", {
-              className: "truncate",
-              children: h
+  return (
+    /*#__PURE__*/
+    // Plain <div> wrapper — Card2 reads as washed-out against the
+    // modal's dark chrome. See ListenersTab + ProvidersTab fix.
+    jsxRuntime.jsxs("div", {
+      className: "w-56 flex-shrink-0 overflow-hidden flex flex-col",
+      children: [/*#__PURE__*/jsxRuntime.jsx("div", {
+        className: "px-3 py-2 text-xs font-semibold opacity-50 uppercase tracking-wider",
+        children: "Event Handlers"
+      }), /*#__PURE__*/jsxRuntime.jsx(DashReact.Divider, {}), /*#__PURE__*/jsxRuntime.jsx("div", {
+        className: "overflow-y-auto flex-1",
+        children: receiver.eventHandlers.length === 0 ? /*#__PURE__*/jsxRuntime.jsx("div", {
+          className: "text-xs opacity-50 text-center py-6 px-3",
+          children: "This widget declares no event handlers."
+        }) : receiver.eventHandlers.map(function (h) {
+          var isActive = h === selectedHandler;
+          var count = countsByHandler.get(h) || 0;
+          return /*#__PURE__*/jsxRuntime.jsxs("button", {
+            type: "button",
+            onClick: function onClick() {
+              return onSelectHandler(h);
+            },
+            className: "w-full text-left px-3 py-2 text-sm flex items-center justify-between gap-2 border-l-2 ".concat(isActive ? "bg-indigo-900/30 border-indigo-400" : "border-transparent hover:bg-white/5"),
+            children: [/*#__PURE__*/jsxRuntime.jsxs("span", {
+              className: "flex items-center gap-2 min-w-0",
+              children: [/*#__PURE__*/jsxRuntime.jsx(DashReact.FontAwesomeIcon, {
+                icon: "bolt",
+                className: "h-3 w-3 opacity-60 flex-shrink-0"
+              }), /*#__PURE__*/jsxRuntime.jsx("code", {
+                className: "truncate",
+                children: h
+              })]
+            }), /*#__PURE__*/jsxRuntime.jsx("span", {
+              className: "text-xs opacity-60 flex-shrink-0",
+              children: count
             })]
-          }), /*#__PURE__*/jsxRuntime.jsx("span", {
-            className: "text-xs opacity-60 flex-shrink-0",
-            children: count
-          })]
-        }, h);
-      })
-    })]
-  });
+          }, h);
+        })
+      })]
+    })
+  );
 }
 
 /**
@@ -60194,10 +60186,7 @@ function EventsColumn(_ref11) {
     return w.isOrphan;
   });
   if (!handlerName) {
-    return /*#__PURE__*/jsxRuntime.jsx(DashReact.Card2, {
-      padding: "",
-      shadow: "",
-      rounded: "rounded-lg",
+    return /*#__PURE__*/jsxRuntime.jsx("div", {
       className: "flex-1 min-w-0 flex items-center justify-center text-sm opacity-60",
       children: "Select a handler to view available events."
     });
@@ -60226,63 +60215,64 @@ function EventsColumn(_ref11) {
   var emittersForList = emitters.filter(function (e) {
     return !(e.component === receiver.component && String(e.itemId) === String(receiver.itemId));
   });
-  return /*#__PURE__*/jsxRuntime.jsxs(DashReact.Card2, {
-    padding: "",
-    shadow: "",
-    rounded: "rounded-lg",
-    className: "flex-1 min-w-0 overflow-hidden flex flex-col",
-    children: [/*#__PURE__*/jsxRuntime.jsxs("div", {
-      className: "flex-shrink-0 px-4 py-2 text-xs opacity-60",
-      children: ["Check an event to fire ", /*#__PURE__*/jsxRuntime.jsx("code", {
-        className: "text-xs",
-        children: handlerName
-      }), " on", " ", /*#__PURE__*/jsxRuntime.jsx("span", {
-        className: "font-medium",
-        children: receiver.label
-      }), "."]
-    }), /*#__PURE__*/jsxRuntime.jsx(DashReact.Divider, {}), /*#__PURE__*/jsxRuntime.jsx("div", {
-      className: "flex-1 overflow-y-auto p-4 space-y-5",
-      children: emittersForList.length === 0 ? /*#__PURE__*/jsxRuntime.jsx("div", {
-        className: "text-sm opacity-60",
-        children: "No other widgets in this dashboard emit events."
-      }) : emittersForList.map(function (e) {
-        return /*#__PURE__*/jsxRuntime.jsxs("div", {
-          className: "space-y-1",
-          children: [/*#__PURE__*/jsxRuntime.jsxs("div", {
-            className: "flex flex-col gap-0.5 mb-1",
-            children: [/*#__PURE__*/jsxRuntime.jsx("div", {
-              className: "text-sm font-semibold",
-              children: e.label
-            }), (e.widgetRef || e.component) && /*#__PURE__*/jsxRuntime.jsx(DashReact.Caption, {
-              block: true,
-              className: "font-mono truncate",
-              children: (e.widgetRef || e.component) + (e.itemId != null ? "[".concat(e.itemId, "]") : "")
-            })]
-          }), e.events.map(function (eventName) {
-            var key = "".concat(e.component, "|").concat(e.itemId, "|").concat(eventName);
-            var selected = wiredKeys.has(key);
-            return /*#__PURE__*/jsxRuntime.jsxs("label", {
-              className: "flex items-center gap-3 px-3 py-1.5 rounded-md cursor-pointer text-sm ".concat(selected ? "opacity-100" : "opacity-70 hover:opacity-100"),
-              children: [/*#__PURE__*/jsxRuntime.jsx(DashReact.FontAwesomeIcon, {
-                icon: selected ? "square-check" : "square",
-                className: "h-4 w-4 flex-shrink-0",
-                onClick: function onClick(ev) {
-                  ev.preventDefault();
-                  toggle({
-                    sourceComponent: e.component,
-                    sourceItemId: e.itemId,
-                    eventName: eventName
-                  });
-                }
-              }), /*#__PURE__*/jsxRuntime.jsx("span", {
-                children: eventName
+  return (
+    /*#__PURE__*/
+    // Plain <div> — Card2's washed-out fill clashes with the modal chrome.
+    jsxRuntime.jsxs("div", {
+      className: "flex-1 min-w-0 overflow-hidden flex flex-col",
+      children: [/*#__PURE__*/jsxRuntime.jsxs("div", {
+        className: "flex-shrink-0 px-4 py-2 text-xs opacity-60",
+        children: ["Check an event to fire ", /*#__PURE__*/jsxRuntime.jsx("code", {
+          className: "text-xs",
+          children: handlerName
+        }), " on", " ", /*#__PURE__*/jsxRuntime.jsx("span", {
+          className: "font-medium",
+          children: receiver.label
+        }), "."]
+      }), /*#__PURE__*/jsxRuntime.jsx(DashReact.Divider, {}), /*#__PURE__*/jsxRuntime.jsx("div", {
+        className: "flex-1 overflow-y-auto p-4 space-y-5",
+        children: emittersForList.length === 0 ? /*#__PURE__*/jsxRuntime.jsx("div", {
+          className: "text-sm opacity-60",
+          children: "No other widgets in this dashboard emit events."
+        }) : emittersForList.map(function (e) {
+          return /*#__PURE__*/jsxRuntime.jsxs("div", {
+            className: "space-y-1",
+            children: [/*#__PURE__*/jsxRuntime.jsxs("div", {
+              className: "flex flex-col gap-0.5 mb-1",
+              children: [/*#__PURE__*/jsxRuntime.jsx("div", {
+                className: "text-sm font-semibold",
+                children: e.label
+              }), (e.widgetRef || e.component) && /*#__PURE__*/jsxRuntime.jsx(DashReact.Caption, {
+                block: true,
+                className: "font-mono truncate",
+                children: (e.widgetRef || e.component) + (e.itemId != null ? "[".concat(e.itemId, "]") : "")
               })]
-            }, key);
-          })]
-        }, e.key || "".concat(e.component, "|").concat(e.itemId));
-      })
-    })]
-  });
+            }), e.events.map(function (eventName) {
+              var key = "".concat(e.component, "|").concat(e.itemId, "|").concat(eventName);
+              var selected = wiredKeys.has(key);
+              return /*#__PURE__*/jsxRuntime.jsxs("label", {
+                className: "flex items-center gap-3 px-3 py-1.5 rounded-md cursor-pointer text-sm ".concat(selected ? "opacity-100" : "opacity-70 hover:opacity-100"),
+                children: [/*#__PURE__*/jsxRuntime.jsx(DashReact.FontAwesomeIcon, {
+                  icon: selected ? "square-check" : "square",
+                  className: "h-4 w-4 flex-shrink-0",
+                  onClick: function onClick(ev) {
+                    ev.preventDefault();
+                    toggle({
+                      sourceComponent: e.component,
+                      sourceItemId: e.itemId,
+                      eventName: eventName
+                    });
+                  }
+                }), /*#__PURE__*/jsxRuntime.jsx("span", {
+                  children: eventName
+                })]
+              }, key);
+            })]
+          }, e.key || "".concat(e.component, "|").concat(e.itemId));
+        })
+      })]
+    })
+  );
 }
 
 function ownKeys$9(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }