Modify

Opened 8 years ago

Closed 6 years ago

Last modified 6 years ago

#1459 closed Bug/Fehler (fixed)

Überarbeitung der favicons

Reported by: Torsten Riemer Owned by: Gerhard Waldemair
Priority: hoch Milestone: modified-shop-2.0.5.0
Component: Module Version: 2.0.3.0
Keywords: Cc:
Blocked By: Blocking:

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 by Torsten Riemer, 8 years ago

Milestone: modified-shop-2.0.4.0modified-shop-2.0.4.1

comment:2 by Torsten Riemer, 7 years ago

Milestone: modified-shop-2.0.4.1modified-shop-2.0.5.0

comment:3 by voodoopupp, 7 years ago

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 by Torsten Riemer, 7 years ago

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

comment:5 by voodoopupp, 7 years ago

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

Version 0, edited 7 years ago by voodoopupp (next)

comment:6 by Torsten Riemer, 7 years ago

comment:7 by voodoopupp, 7 years ago

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 by Gerhard Waldemair, 6 years ago

Owner: set to Gerhard Waldemair
Resolution: fixed
Status: newclosed

In 11925:

fix #1251 & #1459

comment:9 by Torsten Riemer, 6 years ago

In 11929:

fix #1459

comment:10 by Markus, 6 years ago

In 11932:

fix #1459

comment:11 by Markus, 6 years ago

In 11939:

fix #1459 - add favicons

Modify Ticket

Action
as closed The owner will remain Gerhard Waldemair.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.