@visitwonders/assembly 0.16.1 → 0.17.0

package/dist/overlay/popover.js

@@ -245,11 +245,14 @@ class PopoverClose extends Component {
 // ============================================================================
 class Popover extends Component {
   static {
-    g(this.prototype, "_isOpenInternal", [tracked], function () {
-      return undefined;
+    g(this.prototype, "_isOpen", [tracked], function () {
+      return false;
     });
   }
-  #_isOpenInternal = (i(this, "_isOpenInternal"), void 0);
+  #_isOpen = (i(this, "_isOpen"), void 0); // Self-managed open state. The popover opens and closes itself on click,
+  // hover, or focus (per `@trigger`); this is the component's own UI
+  // mechanics, the ember-basic-dropdown shape. `@open`, when provided,
+  // overrides it.
   static {
     g(this.prototype, "triggerElement", [tracked], function () {
       return null;
@@ -307,20 +310,13 @@ class Popover extends Component {
       });
     };
   });
-  get isOpenInternal() {
-    if (this._isOpenInternal === undefined) {
-      return this.args.defaultOpen ?? false;
-    }
-    return this._isOpenInternal;
-  }
-  set isOpenInternal(value) {
-    this._isOpenInternal = value;
-  }
-  get isControlled() {
+  // True when the consumer is driving open via `@open`. The override wins
+  // over the self-managed `_isOpen` state.
+  get hasOpenOverride() {
     return this.args.open !== undefined;
   }
   get isOpen() {
-    return this.isControlled ? this.args.open : this.isOpenInternal;
+    return this.args.open ?? this._isOpen;
   }
   get triggerMode() {
     return this.args.trigger ?? 'click';
@@ -338,8 +334,11 @@ class Popover extends Component {
     return this.args.closeDelay ?? 0;
   }
   setOpen = value => {
-    if (!this.isControlled) {
-      this.isOpenInternal = value;
+    // Update the self-managed state unless the consumer is driving `@open`.
+    // The callback fires either way so an `@open` consumer can follow the
+    // popover's own open/close intent rather than fighting it.
+    if (!this.hasOpenOverride) {
+      this._isOpen = value;
     }
     this.args.onOpenChange?.(value);
   };

package/dist/overlay/tooltip.js

@@ -67,11 +67,13 @@ class TooltipProvider extends Component {
  */
 class Tooltip extends Component {
   static {
-    g(this.prototype, "_isOpenInternal", [tracked], function () {
-      return undefined;
+    g(this.prototype, "_isOpen", [tracked], function () {
+      return false;
     });
   }
-  #_isOpenInternal = (i(this, "_isOpenInternal"), void 0);
+  #_isOpen = (i(this, "_isOpen"), void 0); // Self-managed open state. The tooltip opens and closes itself on hover and
+  // focus; this is the component's own UI mechanics, the ember-basic-dropdown
+  // shape. `@open`, when provided, overrides it.
   static {
     g(this.prototype, "actualSide", [tracked], function () {
       return 'top';
@@ -92,21 +94,14 @@ class Tooltip extends Component {
   // ============================================================================
   // Getters
   // ============================================================================
-  get isOpenInternal() {
-    if (this._isOpenInternal === undefined) {
-      return this.args.defaultOpen ?? false;
-    }
-    return this._isOpenInternal;
-  }
-  set isOpenInternal(value) {
-    this._isOpenInternal = value;
-  }
-  get isControlled() {
+  // True when the consumer is driving open via `@open`. The override wins
+  // over the self-managed `_isOpen` state.
+  get hasOpenOverride() {
     return this.args.open !== undefined;
   }
   get isOpen() {
     if (this.args.disabled) return false;
-    return this.isControlled ? this.args.open : this.isOpenInternal;
+    return this.args.open ?? this._isOpen;
   }
   get side() {
     return this.args.side ?? 'top';
@@ -244,8 +239,11 @@ class Tooltip extends Component {
   // State Management
   // ============================================================================
   setOpen = value => {
-    if (!this.isControlled) {
-      this.isOpenInternal = value;
+    // Update the self-managed state unless the consumer is driving `@open`.
+    // The callback fires either way so an `@open` consumer can follow the
+    // tooltip's own open/close intent rather than fighting it.
+    if (!this.hasOpenOverride) {
+      this._isOpen = value;
     }
     this.args.onOpenChange?.(value);
     // Track close time for skip delay behavior

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@visitwonders/assembly",
-  "version": "0.16.1",
+  "version": "0.17.0",
   "description": "The default blueprint for Embroider v2 addons.",
   "keywords": [
     "ember-addon"