From 7252c14a1679be4d758f79ef73b8dc4563a7b470 Mon Sep 17 00:00:00 2001
From: Penar Musaraj <pmusaraj@gmail.com>
Date: Wed, 11 Sep 2019 00:18:12 -0400
Subject: [PATCH] FIX: Switch to full screen external login for Safari

This is a temporary solution, currently popup auth fails in Safari.
---
 app/assets/javascripts/discourse/controllers/login.js.es6 | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/app/assets/javascripts/discourse/controllers/login.js.es6 b/app/assets/javascripts/discourse/controllers/login.js.es6
index 19da455a26ed06..13c77deab76a56 100644
--- a/app/assets/javascripts/discourse/controllers/login.js.es6
+++ b/app/assets/javascripts/discourse/controllers/login.js.es6
@@ -217,7 +217,10 @@ export default Ember.Controller.extend(ModalFunctionality, {
       // On Mobile, Android or iOS always go with full screen
       if (
         this.isMobileDevice ||
-        (capabilities && (capabilities.isIOS || capabilities.isAndroid))
+        (capabilities &&
+          (capabilities.isIOS ||
+            capabilities.isAndroid ||
+            capabilities.isSafari))
       ) {
         fullScreenLogin = true;
       }