Favicon Erweiterung unterstützt jetzt auch PNG und GIF

[jan.theofel]

Hallo,

laut Spezifikation dürfen auch PNG und GIF Dateien als Favicon genutzt werden. Dies unterstützt die Erweiterung Favicon ab Version 1.1.1 nun auf Anforderung von Andreas Schempp.

Jan

[andreasisaak]

Leider zeigt der IE aber keine PNG und GIF Favicons an

[Andreas]

Dann hilft vielleicht ein Fallback

<link href="favicon.ico" rel="shortcut icon" type="image/x-icon" />
<link href="animated_favicon.gif" rel="icon" type="image/gif" />
<link href="apple-touch-icon.png" rel="apple-touch-icon" />

Hab ich hierher

http://www.html-kit.com/favicon/

Grüße

[Dani]

Hallo ETES

Wäre es möglich das ", PageRegular $objPageRegular" aus der createFavicon()-Funktion herauszulöschen? Denn ich habe die PageRegular zu PageRegularExtended erweitert und beim Aufruf deiner Funktion gibt es mir einen Instanz-Error im FE, ohne nicht. Und wie ich sehe, wird dieser Parameter gar nicht benötigt.

Ich danke dir!

[jan.theofel]

Hi Dani,

ich nutze hier den HOOK aus dem Core. Solange du keine Anpassungen im Core vornimmst (was du nicht tun solltest), sollte es da keine Probleme geben.

Jan

[Dani]

Ich habe aber Änderungen am Code vorgenommen, da es nicht anders ging.
Den Parameter braucht es gar nicht, wenn du ihn gar nicht in der Funktion verwendest. Ich denke auch, dass es die Ladezeit minim verkürzen würde, wenn du ihn rausnehmen würdest. Es gibt ja keinen wirklichen Grund, ihn in der Funktion drin zu behalten.

Wäre es nicht möglich in rauszunehmen?

http://www.contao.org/hooks-verwenden.html#generatePage

[jan.theofel]

Hi dani,

Ich habe aber Änderungen am Code vorgenommen, da es nicht anders ging.

Dann solltest du Leo ein Ticket machen, wo ein Hook fehlt, so dass du das in Zukunft updatesicher hinbekommst. Oder offenlegen was du anpassen willst - vielleicht geht es ja doch ohne Core-Anpassung.

Den Parameter braucht es gar nicht, wenn du ihn gar nicht in der Funktion verwendest. Ich denke auch, dass es die Ladezeit minim verkürzen würde, wenn du ihn rausnehmen würdest. Es gibt ja keinen wirklichen Grund, ihn in der Funktion drin zu behalten.

Der Core übergibt nun mal diese Parameter. Es ist nicht der Sinn von Spezifikationen (welche Hooks bekommen welche Parameter) sich über sie hinwegzusetzen, weil es "ja auch so tut". Und das es jetzt fehlerfrei läuft bedeutet nicht, dass es das überall und auch in der nächsten Version von Contao tun wird. Und dann darf ich mich ggf. wieder mit Fehlern rumschlagen. Von daher musst du leider neben dem Core auch das Modul patchen. Sorry.

Jan

[Dani]

Dann solltest du Leo ein Ticket machen, wo ein Hook fehlt, so dass du das in Zukunft updatesicher hinbekommst. Oder offenlegen was du anpassen willst - vielleicht geht es ja doch ohne Core-Anpassung.

Habe ich schon.

[Dani]

Ich konnte das Favicon-Problem jetzt trotzdem anders lösen.

[jan.theofel]

Hi zusammen,

<link href="apple-touch-icon.png" rel="apple-touch-icon" />

Ich habe gerade die Version 1.3.0 online gestellt, die Unterstützung für das Webclip-Icon mit enthält.

Jan

[alex]

Hallo Jan,

warum hast Du denn "Webclip-Icon" und "Favicon" als "Pflicht-Angaben hinterlegt. Ich fände eine Variante die besser, die beide Einträge prüft und je nach dem die Ausgabe macht:

        if($favicon != '')
        {
            $GLOBALS['TL_HEAD'][] = '<link rel="icon" type="image/vnd.microsoft.icon" href="'.$favicon.'" />';
            $GLOBALS['TL_HEAD'][] = '<link rel="shortcut icon" type="image/vnd.microsoft.icon" href="'.$favicon.'" />';
        }
        if($appleTouchIcon != '')
        {
            $GLOBALS['TL_HEAD'][] = '<link rel="apple-touch-icon" href="'.$appleTouchIcon.'" />';
        } 

[jan.theofel]

Hi Alex,

danke für den Hinweis. Wir haben gestern noch die korrigierte Version 1.3.1 online gestellt, die diesen Fehler behebt.

Jan