@trops/dash-core 0.1.565 → 0.1.566

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