FIX: Update mobile logo resolution

This makes Discourse compliant with latest Google PWA requirements,
so we get the App Install banner back.

Should bump our Lighthouse PWA Audit score to 11/11.
This commit is contained in:
Rafael dos Santos Silva
2017-11-01 01:51:51 -02:00
parent d888d3c54c
commit 32b3847d52
2 changed files with 5 additions and 3 deletions

View File

@@ -13,6 +13,8 @@ class MetadataController < ApplicationController
private
def default_manifest
logo = SiteSetting.mobile_logo_url || SiteSetting.logo_small_url || SiteSetting.apple_touch_icon_url
manifest = {
name: SiteSetting.title,
short_name: SiteSetting.title,
@@ -23,8 +25,8 @@ class MetadataController < ApplicationController
theme_color: "##{ColorScheme.hex_for_name('header_background')}",
icons: [
{
src: SiteSetting.apple_touch_icon_url,
sizes: "144x144",
src: logo,
sizes: "512x512",
type: "image/png"
}
]