Refactor #12031 - Add two-way binding to visible option

primefaces · Nov 4, 2022 · 6ce9c94 · 6ce9c94
1 parent e397612
commit 6ce9c94
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/app/components/overlay/overlay.ts b/src/app/components/overlay/overlay.ts
@@ -156,6 +156,8 @@ export class Overlay implements OnDestroy {
         this._options = val;
     }
 
+    @Output() visibleChange: EventEmitter<any> = new EventEmitter();
+
     @Output() onBeforeShow: EventEmitter<any> = new EventEmitter();
 
     @Output() onShow: EventEmitter<any> = new EventEmitter();
@@ -252,21 +254,26 @@ export class Overlay implements OnDestroy {
     }
 
     show(overlay?: HTMLElement, isFocus: boolean = false) {
-        this.visible = true;
+        this.onVisibleChange(true);
         this.handleEvents('onShow', { overlay: overlay || this.overlayEl, target: this.targetEl, mode: this.overlayMode });
 
         isFocus && DomHandler.focus(this.targetEl);
         this.modal && DomHandler.addClass(this.document?.body, 'p-overflow-hidden');
     }
 
     hide(overlay?: HTMLElement, isFocus: boolean = false) {
-        this.visible = false;
+        this.onVisibleChange(false);
         this.handleEvents('onHide', { overlay: overlay || this.overlayEl, target: this.targetEl, mode: this.overlayMode });
 
         isFocus && DomHandler.focus(this.targetEl);
         this.modal && DomHandler.removeClass(this.document?.body, 'p-overflow-hidden');
     }
 
+    onVisibleChange(visible: boolean) {
+        this._visible = visible;
+        this.visibleChange.emit(visible);
+    }
+
     alignOverlay() {
         !this.modal && DomHandler.alignOverlay(this.overlayEl, this.targetEl, this.appendTo);
     }