lecodes-sdk 2.5.1 → 2.8.0

package/src/ui/UIWidget.ts

@@ -73,15 +73,25 @@ export class WidgetElement extends ContainerElement<"widget"> {
     // factory) so subclasses like ModalElement inherit it.
     super(type, { position: "absolute", ...style }, children)
   }
-  show(): void {
+  show(): void { this._setOpen(true) }
+  hide(): void { this._setOpen(false) }
+  /** @internal The body of show() / hide(): what every way of opening a widget runs — the calls,
+   *  a modal's `open` binding. Subclasses wrap it (a modal syncs its pose, a sheet its detent). */
+  _setOpen(open: boolean): void {
     if (this._h.id === 0) return
+    if (!open) {
+      if (!this._shown) return
+      this._setShown(false)
+      this._writeDisplay("none")
+      return
+    }
     const parentId = tree().parent(this._h.id)
     if (parentId !== 0 && this._shown) return
     if (parentId === 0) {
       // No place of its own: the app layer takes it HIDDEN, and the flip below shows it — a show()
       // is a display flip wherever the widget is, so its pose plays (a modal's fade) exactly as on
       // a placed one. Appended shown, it would merely appear: a widget shows with its place, no pose.
-      this.style({ display: "none" } as any)
+      this._writeDisplay("none")
       _appLayer().append(this as any)   // pins it: _pinned reads the display
     } else {
       // Shown = on top of its root: a show() moves the widget to the TAIL of its parent's children
@@ -94,13 +104,16 @@ export class WidgetElement extends ContainerElement<"widget"> {
         tree().insert(parentId, this._h.id, siblings.length - 1)
       }
     }
-    this.style({ display: "flex" } as any)
+    this._writeDisplay("flex")
     this._setShown(true)
   }
-  hide(): void {
-    if (this._h.id === 0 || !this._shown) return
-    this._setShown(false)
-    this.style({ display: "none" } as any)
+  /** @internal True while the widget writes its own display (show / hide / open): the one display
+   *  write a modal lets through — the app's `display` on a modal is refused (ModalElement._mapStyle). */
+  _ownWrite = false
+  /** @internal The widget's own display write. */
+  _writeDisplay(display: "flex" | "none"): void {
+    this._ownWrite = true
+    try { this.style({ display } as any) } finally { this._ownWrite = false }
   }
   get isShown(): boolean {
     return this._shown
@@ -115,7 +128,7 @@ export class WidgetElement extends ContainerElement<"widget"> {
     }
     if (owner) {
       // Hidden first: a widget shows with its place, and the old contract showed it on show() only.
-      this.style({ display: "none" } as any)
+      this._writeDisplay("none")
       owner.widgets.append(this as any)
     }
     return this

package/src/version.ts

@@ -4,7 +4,7 @@
 // same manifest). Its MAJOR is the bundle ↔ runtime contract: a bundle's `// sdk:` header line
 // and a host's embedded version must agree on the major for the host to run the bundle; minor
 // and patch never gate anything (features are detected, never versioned).
-export const SDK_VERSION = "2.5.1"
+export const SDK_VERSION = "2.8.0"
 
 /** The major of a semver string, or null when it is not one. */
 export const sdkMajor = (version: string | null | undefined): number | null => {