Merge pull request #272 from microsoftgraph/nmetulev/mgt-login-popup-fix

Fixed issue with mgt-login popup not opening

Author: nmetulev

src/components/mgt-login/mgt-login.ts

@@ -72,6 +72,7 @@ export class MgtLogin extends MgtBaseComponent {
    */
   public connectedCallback() {
     super.connectedCallback();
+    this.addEventListener('click', e => e.stopPropagation());
     window.addEventListener('click', this.handleWindowClick);
   }
 
@@ -199,16 +200,7 @@ export class MgtLogin extends MgtBaseComponent {
   }
 
   private handleWindowClick(e: MouseEvent) {
-    if (e.target === this) {
-      return;
-    }
-
-    // get popup bounds
-    const popup = this.renderRoot.querySelector('.popup');
-    if (popup) {
-      this._popupRect = popup.getBoundingClientRect();
-      this._showMenu = false;
-    }
+    this._showMenu = false;
   }
 
   private renderLogIn() {