From cc17df161d85f50a7d802f2823f7f4cbcec5262c Mon Sep 17 00:00:00 2001
From: Wroud <wrouds@gmail.com>
Date: Sat, 16 May 2020 01:27:40 +0300
Subject: [PATCH] fix(authentication): provider selection

---
 webapp/packages/authentication/src/Dialog/AuthDialog.tsx      | 2 ++
 .../authentication/src/Dialog/AuthDialogController.ts         | 4 +---
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/webapp/packages/authentication/src/Dialog/AuthDialog.tsx b/webapp/packages/authentication/src/Dialog/AuthDialog.tsx
index 3419d53d55..18cbe67015 100644
--- a/webapp/packages/authentication/src/Dialog/AuthDialog.tsx
+++ b/webapp/packages/authentication/src/Dialog/AuthDialog.tsx
@@ -7,6 +7,7 @@
  */
 
 import { observer } from 'mobx-react';
+import { useEffect } from 'react';
 import { useTabState, Tab as BaseTab, TabList } from 'reakit/Tab';
 import styled, { css } from 'reshadow';
 
@@ -101,6 +102,7 @@ export const AuthDialog: DialogComponent<null, null> = observer(
       selectedId: controller.provider?.id,
     });
     tab.selectedId = controller.provider?.id || null;
+    useEffect(() => controller.selectProvider(tab.selectedId!), [tab.selectedId]);
 
     return styled(useStyles(styles))(
       <CommonDialogWrapper
diff --git a/webapp/packages/authentication/src/Dialog/AuthDialogController.ts b/webapp/packages/authentication/src/Dialog/AuthDialogController.ts
index 544c0f816d..960dfef58b 100644
--- a/webapp/packages/authentication/src/Dialog/AuthDialogController.ts
+++ b/webapp/packages/authentication/src/Dialog/AuthDialogController.ts
@@ -102,9 +102,7 @@ export class AuthDialogController implements IInitializableController, IDestruct
       return providerA.label.localeCompare(providerB.label);
     }
 
-    if (providerA.defaultProvider === providerB.defaultProvider) {
-      return 0;
-    } if (providerA.defaultProvider) {
+    if (providerA.defaultProvider) {
       return -1;
     }
     return 1;