Opened 3 years ago

Closed 2 years ago

Last modified 2 years ago

#1459 closed Bug/Fehler (fixed)

Überarbeitung der favicons

Reported by: Tomcraft Owned by: GTB
Priority: hoch Milestone: modified-shop-2.0.5.0
Component: Module Version: 2.0.3.0

Description

Der Code wurde deutlich verschlankt: https://realfavicongenerator.net/

<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="192x192" href="/android-chrome-192x192.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="msapplication-TileImage" content="/mstile-144x144.png">
<meta name="theme-color" content="#ffffff“>

Die manifest.json wurde umbenannt in site.webmanifest

Attachments (0)

Change History (11)

comment:1 Changed 3 years ago by Tomcraft

  • Milestone changed from modified-shop-2.0.4.0 to modified-shop-2.0.4.1

comment:2 Changed 3 years ago by Tomcraft

  • Milestone changed from modified-shop-2.0.4.1 to modified-shop-2.0.5.0

comment:3 Changed 3 years ago by voodoopupp

Ebenso müssten dann auch die Einträge in der .htaccess berücksichtigt werden, siehe z.B. boilerplate:
https://github.com/h5bp/html5-boilerplate

comment:4 Changed 3 years ago by Tomcraft

Kannst du das ein wenig genauer erklären, was da an der .htaccess geändert werden müsste?

comment:5 Changed 3 years ago by voodoopupp

Ohne Gewähr, sollte aber so stimmen:

Das hier

##-- Add Mime Types
<IfModule mod_mime.c>
  AddType application/font-woff2 .woff2
</IfModule>

ändern in

##-- Add Mime Types
<IfModule mod_mime.c>
  AddType application/font-woff2 .woff2
  AddType application/manifest+json .webmanifest
</IfModule>

Das hier

<FilesMatch "\\.(css|woff|woff2|ttf|otf|eot)$">

ändern in

<FilesMatch "\\.(css|woff|woff2|ttf|otf|eot|manifest)$">

Und ich bin mir nicht ganz sicher, aber ebenso sollte wohl auch das hier

<FilesMatch "\\.(js|css|html|htm|php|xml|jpg|jpeg|png|gif|svg|woff|woff2|ttf|otf|eot)$">

geändert werden in

<FilesMatch "\\.(ico|js|css|html|htm|php|xml|jpg|jpeg|png|gif|svg|woff|woff2|ttf|otf|eot|manifest)$">

Und dazu sollte gleich noch Ticket #1530 berücksichtigt werden.

Last edited 3 years ago by Tomcraft (previous) (diff)

comment:6 Changed 3 years ago by Tomcraft

comment:7 Changed 3 years ago by voodoopupp

Richtig, und wenn man in der includes/modules/favicons.php das zweimalige Vorkommen von

manifest.json

ändert in

site.webmanifest

dann wird auch mit den neuen Favicons Version die korrekte Manifest-Datei eingebunden.
Allerdings denke ich, dass ihr irgendwo auch einen Fallback für die alte Version einbinden solltet.

comment:8 Changed 2 years ago by GTB

  • Owner set to GTB
  • Resolution set to fixed
  • Status changed from new to closed

In 11925:

fix #1251 & #1459

comment:9 Changed 2 years ago by Tomcraft

In 11929:

fix #1459

comment:10 Changed 2 years ago by Markus

In 11932:

fix #1459

comment:11 Changed 2 years ago by Markus

In 11939:

fix #1459 - add favicons

Add Comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.