fix(overlay): hover trigger stop fire show if container already exists (

#947)
akveo · Nov 14, 2018 · 2988a4b · 2988a4b
1 parent 57777f8
commit 2988a4b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/framework/theme/components/cdk/overlay/overlay-trigger.spec.ts b/src/framework/theme/components/cdk/overlay/overlay-trigger.spec.ts
@@ -101,7 +101,9 @@ describe('hover-trigger-strategy', () => {
   });
 
   it('should fire show$ when hover on host', done => {
-    const triggerStrategy = triggerStrategyBuilder.build();
+    const triggerStrategy = triggerStrategyBuilder
+      .container(() => null)
+      .build();
     triggerStrategy.show$.subscribe(done);
     mouseEnter(host);
   });

diff --git a/src/framework/theme/components/cdk/overlay/overlay-trigger.ts b/src/framework/theme/components/cdk/overlay/overlay-trigger.ts
@@ -84,6 +84,7 @@ export class NbHoverTriggerStrategy extends NbTriggerStrategy {
 
   show$: Observable<Event> = observableFromEvent<Event>(this.host, 'mouseenter')
     .pipe(
+      filter(() => !this.container()),
       delay(100),
       takeUntil(observableFromEvent(this.host, 'mouseleave')),
       repeat(),