@vectoriox/iox-builder 4.35.4 → 4.35.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -8489,10 +8489,21 @@ class BuilderDialogComponent {
8489
8489
  this.designOpen = false;
8490
8490
  }
8491
8491
  ngOnInit() {
8492
- // A freshly dropped EMPTY dialog opens itself — "lands open" (board A4).
8493
- this.designOpen = this.children.length === 0;
8494
8492
  // Board A4: "Closed later? Click it [in the tree] and it reopens." Selection IS the
8495
8493
  // reopen signal — tree row, breadcrumb crumb, target-picker jump all land here.
8494
+ //
8495
+ // ⚠️ It is ALSO the lands-open signal. This used to read
8496
+ // `designOpen = this.children.length === 0`, on the premise that `ngOnInit` means "just
8497
+ // dropped". It doesn't — it runs on every page LOAD too, so any saved page holding an
8498
+ // empty Dialog opened its modal stage the moment it opened, and the stage is
8499
+ // `position: fixed; inset: 0`: it swallowed every click on the canvas until somebody
8500
+ // thought to click the backdrop. That is precisely the "a closed overlay has NO canvas
8501
+ // presence" rule, broken by the component that the rule was written for.
8502
+ //
8503
+ // A fresh drop is distinguishable without a new flag: `_selectAfterDrop` selects the new
8504
+ // node a tick after it renders, and `select$` is a BehaviorSubject — so a dialog dropped
8505
+ // right now still lands open, through the line below, while one that was merely loaded
8506
+ // stays shut because nothing is selected on load.
8496
8507
  this.selectSub = this.overlayService.select$?.subscribe?.((sel) => {
8497
8508
  const id = sel?.node?.id ?? sel?.nodeId;
8498
8509
  if (id && id === this.nodeId && !this.designOpen) {