Merge pull request #8300 from IgniteUI/mvenkov/handle_correctly_target_as_point-9.1

Handle correctly target as point

Author: kdinev

projects/igniteui-angular/src/lib/services/overlay/overlay.spec.ts

@@ -881,6 +881,35 @@ describe('igxOverlay', () => {
             expect(element.style.width).toBe('200px');
             expect(element.style.height).toBe('100px');
         });
+
+        it('should close overlay on outside click when target is point, #8297', fakeAsync(() => {
+            const fix = TestBed.createComponent(EmptyPageComponent);
+            const button = fix.componentInstance.buttonElement;
+            const overlay = fix.componentInstance.overlay;
+            fix.detectChanges();
+
+            const overlaySettings: OverlaySettings = {
+                modal: false,
+                closeOnOutsideClick: true,
+                positionStrategy: new ConnectedPositioningStrategy()
+            };
+
+            overlaySettings.positionStrategy.settings.target = new Point(10, 10);
+
+            overlay.show(overlay.attach(SimpleDynamicComponent), overlaySettings);
+            tick();
+            fix.detectChanges();
+
+            let overlayDiv: Element = document.getElementsByClassName(CLASS_OVERLAY_MAIN)[0];
+            expect(overlayDiv).toBeDefined();
+
+            document.body.click();
+            tick();
+            fix.detectChanges();
+
+            overlayDiv = document.getElementsByClassName(CLASS_OVERLAY_MAIN)[0];
+            expect(overlayDiv).toBeUndefined();
+        }));
     });
 
     describe('Unit Tests - Scroll Strategies: ', () => {

projects/igniteui-angular/src/lib/services/overlay/overlay.ts

@@ -660,7 +660,7 @@ export class IgxOverlayService implements OnDestroy {
                     // if we should exclude position target check if the click is over it. If so do not close overlay
                     const positionTarget = info.settings.positionStrategy.settings.target as HTMLElement;
                     let clickOnPositionTarget = false;
-                    if (positionTarget) {
+                    if (positionTarget && positionTarget.contains) {
                         clickOnPositionTarget = positionTarget.contains(target);
                     }