search

processwire / processwire-issues

ProcessWire issue reports.
45 stars 2 forks source link

processwire.com favicons #1306

Open adrianbj opened 3 years ago

adrianbj commented 3 years ago

Short description of the issue

Currently there isn't a full set of appropriate favicons for the site so you end up with things looking like this:

image

Optional: Suggestion for a possible fix

Here is a full set icons created from a clean SVG version of the logo and the appropriate code for the various apple / MS etc versions.

<link rel="apple-touch-icon" sizes="180x180" href="/apple-touch-icon.png">
<link rel="icon" type="image/png" sizes="32x32" href="/favicon-32x32.png">
<link rel="icon" type="image/png" sizes="16x16" href="/favicon-16x16.png">
<link rel="manifest" href="/site.webmanifest">
<link rel="mask-icon" href="/safari-pinned-tab.svg" color="#5bbad5">
<meta name="msapplication-TileColor" content="#da532c">
<meta name="theme-color" content="#ffffff">

favicon_package.zip

ryancramerdesign commented 3 years ago

@adrianbj Thanks Adrian! We have something similar in the site already, so I'm wondering why it might not be working in the case you mentioned. This is what we currently have below. Are you able to tell which one wasn't working? They all seem to work in my tests here, so I'm not sure. 

<link rel="apple-touch-icon" sizes="180x180" href="https://processwire.com/site/favicon/pw180.png"><?php /* apple touch icon */ ?>
<link rel="icon" type="image/png" sizes="192x192" href="https://processwire.com/site/favicon/pw192.png">
<link rel="icon" type="image/png" sizes="32x32" href="https://processwire.com/site/favicon/pw32.png">
<link rel="icon" type="image/png" sizes="16x16" href="https://processwire.com/site/favicon/pw16.png">
<link rel="manifest" href="https://processwire.com/site/favicon/site.webmanifest"><?php /* android */ ?>
<link rel="mask-icon" href="https://processwire.com/site/favicon/pw.svg" color="#2480e6"><?php /* safari pinned tab */ ?>

<meta name="apple-mobile-web-app-title" content="ProcessWire CMS">
<meta name="application-name" content="ProcessWire CMS">
<meta name="msapplication-TileColor" content="#2b5797">
<meta name="theme-color" content="#ffffff">
adrianbj commented 3 years ago

I guess think it's just the forum site actually. I am a Brave user and it adds commonly used site icons to the new tab screen and it looks like that for the https://processwire.com/talk/ link.

ryancramerdesign commented 3 years ago

Yes indeed you are a brave user, thank you for your continued courage and support. If it's happening in the forums, perhaps the forums are overriding them with their own set, I have a look at that, but might need Pete's help to figure out exactly where this should be fixed.