Skip to content
This repository was archived by the owner on Jan 6, 2025. It is now read-only.

Commit 942295d

Browse files
committed
fixup! fix(show-hide): account for fxLayout on same element
1 parent 63c0d17 commit 942295d

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

src/lib/extended/show-hide/show-hide.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ export class ShowHideDirective extends BaseDirective2 implements AfterViewInit,
8080
this.hasLayout = this.marshal.hasValue(this.nativeElement, 'layout');
8181
this.marshal.trackValue(this.nativeElement, 'layout')
8282
.pipe(takeUntil(this.destroySubject))
83-
.subscribe(this.onLayoutChange.bind(this));
83+
.subscribe(this.triggerUpdate.bind(this));
8484

8585
const children = Array.from(this.nativeElement.children);
8686
for (let i = 0; i < children.length; i++) {
@@ -143,11 +143,6 @@ export class ShowHideDirective extends BaseDirective2 implements AfterViewInit,
143143
'flex' : this.styler.lookupStyle(this.nativeElement, 'display', true);
144144
}
145145

146-
/** Respond to layout changes on host element by reactivating show-hide */
147-
protected onLayoutChange() {
148-
this.updateWithValue(this.marshal.getValue(this.nativeElement, this.DIRECTIVE_KEY));
149-
}
150-
151146
/** Validate the visibility value and then update the host's inline display style */
152147
protected updateWithValue(value: boolean|string = true) {
153148
if (value === '') {

0 commit comments

Comments
 (0)