Browse Source

Fix invalid `mask-icon` when a custom instance icon is configured (#30734)

dariusk-working/4_3_0
Renaud Chaput 2 years ago committed by GitHub
parent
commit
43bbdea421
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
  1. 4
      app/helpers/application_helper.rb
  2. 3
      app/views/layouts/application.html.haml

4
app/helpers/application_helper.rb

@ -256,6 +256,10 @@ module ApplicationHelper
instance_presenter.app_icon&.file&.url(size)
end
def use_mask_icon?
instance_presenter.app_icon.blank?
end
private
def storage_host_var

3
app/views/layouts/application.html.haml

@ -19,7 +19,8 @@
- SiteUpload::APPLE_ICON_SIZES.each do |size|
%link{ rel: 'apple-touch-icon', sizes: "#{size}x#{size}", href: app_icon_path(size.to_i) || frontend_asset_path("icons/apple-touch-icon-#{size}x#{size}.png") }/
%link{ rel: 'mask-icon', href: frontend_asset_path('images/logo-symbol-icon.svg'), color: '#6364FF' }/
- if use_mask_icon?
%link{ rel: 'mask-icon', href: frontend_asset_path('images/logo-symbol-icon.svg'), color: '#6364FF' }/
%link{ rel: 'manifest', href: manifest_path(format: :json) }/
= theme_color_tags current_theme
%meta{ name: 'apple-mobile-web-app-capable', content: 'yes' }/

Loading…
Cancel
Save