From c8b5e6baae17bbb50a8173b5cd32944df0bad20a Mon Sep 17 00:00:00 2001
From: Rafael dos Santos Silva <xfalcox@gmail.com>
Date: Wed, 15 Aug 2018 23:36:08 -0300
Subject: [PATCH] FEATURE: Use `display: browser` in webmanifest for iOS
 devices

Since iOS doesn't have a back button and can have issues on log in.

See https://twitter.com/firt/status/1021477243909033984
---
 app/controllers/metadata_controller.rb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/controllers/metadata_controller.rb b/app/controllers/metadata_controller.rb
index bd77bd191d..2305936dc6 100644
--- a/app/controllers/metadata_controller.rb
+++ b/app/controllers/metadata_controller.rb
@@ -19,10 +19,12 @@ class MetadataController < ApplicationController
     end
     file_info = get_file_info(logo)
 
+    display = request.user_agent =~ /iPad|iPhone/ ? 'browser' : 'standalone'
+
     manifest = {
       name: SiteSetting.title,
       short_name: SiteSetting.title,
-      display: 'standalone',
+      display: display,
       orientation: 'any',
       start_url: Discourse.base_uri.present? ? "#{Discourse.base_uri}/" : '.',
       background_color: "##{ColorScheme.hex_for_name('secondary')}",