FIX: Switch to full screen external login for Safari

This is a temporary solution, currently popup auth fails in Safari.
discourse · Sep 11, 2019 · 7252c14 · 7252c14
1 parent 93b0ffc
commit 7252c14
Showing 1 changed file with 4 additions and 1 deletion.
diff --git 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;
       }