search

yakamara / redaxo_yrewrite

A multidomain URL rewrite engine for REDAXO
MIT License
56 stars 36 forks source link

Add og:image tags #477

Closed TobiasKrais closed 2 years ago

alxndr-w commented 2 years ago

Finde ich nach wie vor nicht gut, dass Bilder so downgesampelt werden. Das ist nicht Sinn der Sache.

skerbis commented 2 years ago

rex_media_medium ist kein geeignetes Format. Besser einen leeren Medientyp anlegen

TobiasKrais commented 2 years ago

@gharlan über diesen PR herrscht Einigkeit. Wir übergeben an dich.

tbaddade commented 2 years ago

inde ich nach wie vor nicht gut, dass Bilder so downgesampelt werden.

Ich habe die Erfahrung gemacht, wenn Bilder zu groß sind, dass diese beim Teilen nicht akzeptiert werden.

TobiasKrais commented 2 years ago

Also doch noch keine Einigkeit. @alxndr-w und @skerbis: Was würdet ihr davon halten, wenn wir den neuen MM Typen in der Voreinstellung skalieren, sagen wir mal auf 1000px? Die Nutzer können das ja nach Bedarf anpassen.

gharlan commented 2 years ago

Hmm, ich habe jetzt auch mal bisschen gesucht, und keine klare Empfehlung gefunden für die Größe. Facebook spricht zum Beispiel eher von Mindestgrößen bzgl. Pixel, aber es gibt eine Max-Größe bzgl. Dateigröße (8 MB): https://developers.facebook.com/docs/sharing/webmasters/images/

gharlan commented 2 years ago

Ich denke, ich würde es so lassen, wie es hier im PR ist. Also keine Default-Effekte in dem MM-Typ.

TobiasKrais commented 2 years ago

Ich hab grad noch eine Anfrage bei einem hauptberuflichen SEO Experten laufen. Er sagt, dass er sich heute Mittag meldet. Ich gebe Bescheid, wenn ich die Antwort habe.

skerbis commented 2 years ago

zu Twitter:

Images must be less than 5MB in size , Images for this Card support an aspect ratio of 2:1 with minimum dimensions of 300x157 or maximum of 4096x4096 pixels. Images must be less than 5MB in size. JPG, PNG, WEBP and GIF formats are supported.

https://developer.twitter.com/en/docs/twitter-for-websites/cards/overview/summary-card-with-large-image

TobiasKrais commented 2 years ago

Danke @skerbis für diesen wertvollen Hinweis. Ich habe inzwischen meinen SEO Experten befragt. Ihm sind keine weiteren Formatbeschränkungen bekannt. Sollen wir dann einen Effekt mit maximal 4096x4096px hinzufügen?

gharlan commented 2 years ago

Ups sorry, mir fällt gerade auf, dass der 4096px-Vorschlag ja noch offen war. Eigentlich hatte ich den gelesen. Aber gestern Abend habe ich dann doch nicht mehr dran gedacht und war irgendwie der Meinung, der PR wäre fertig. Also sorry. Von meiner Seite aus können die 4096px gerne noch eingebaut werden. Wegen mir müssen sie aber auch nicht rein. Mir ist es eigentlich egal.

TobiasKrais commented 2 years ago

Kein Problem. Ich mach einen neuen PR. Kommt nachher.

TobiasKrais commented 2 years ago

So, PR ist gemacht: https://github.com/yakamara/redaxo_yrewrite/pull/486