From 5ead60677a25c4b5fa10d854d922f8a89ee15ba9 Mon Sep 17 00:00:00 2001 From: Rafael dos Santos Silva Date: Tue, 4 Feb 2020 14:16:00 -0300 Subject: [PATCH] FEATURE: Ensure we always fill the short_name in the web manifest --- app/controllers/metadata_controller.rb | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/app/controllers/metadata_controller.rb b/app/controllers/metadata_controller.rb index 279bdfea1ba..ccde4cd1b79 100644 --- a/app/controllers/metadata_controller.rb +++ b/app/controllers/metadata_controller.rb @@ -35,6 +35,7 @@ class MetadataController < ApplicationController manifest = { name: SiteSetting.title, + short_name: SiteSetting.short_title.presence || SiteSetting.title.truncate(12, separator: ' ', omission: ''), display: display, start_url: Discourse.base_uri.present? ? "#{Discourse.base_uri}/" : '.', background_color: "##{ColorScheme.hex_for_name('secondary', view_context.scheme_id)}", @@ -64,8 +65,6 @@ class MetadataController < ApplicationController manifest[:icons] << icon_entry end - manifest[:short_name] = SiteSetting.short_title if SiteSetting.short_title.present? - if current_user && current_user.trust_level >= 1 && SiteSetting.native_app_install_banner_android manifest = manifest.merge( prefer_related_applications: true,