Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No WebP preview available in filebrowser #503

Closed
frase-git opened this issue May 28, 2021 · 7 comments
Closed

No WebP preview available in filebrowser #503

frase-git opened this issue May 28, 2021 · 7 comments
Labels
Milestone

Comments

@frase-git
Copy link
Contributor

It would be time to support the newer image formats with better compression at good quality by CMSimple_XH - especially *.webp. Theoretically it is already possible to use these formats, but you don't get to see an image preview.

@cmb69
Copy link
Member

cmb69 commented Jun 1, 2021

Das überlappt mit #294, aber da ist insgesamt mehr zu berücksichtigen. Ich bin einfach mal so frei und ändere den Titel, und stufe die fehlende WebP-Vorschau als Bug ein, der noch in XH 1.7.5 gelöst werden sollte.

@cmb69 cmb69 changed the title Support for modern image formats No WebP preview available in filebrowser Jun 1, 2021
@cmb69 cmb69 added the bug label Jun 1, 2021
@cmb69 cmb69 added this to the 1.7.5 milestone Jun 1, 2021
@cmb69
Copy link
Member

cmb69 commented Jun 1, 2021

To clarify: this fix will only work as of PHP 7.0.10.

@cmb69 cmb69 closed this as completed in 9e44be4 Jun 1, 2021
cmb69 added a commit that referenced this issue Jun 1, 2021
* 1.7:
  Fix #503: No WebP preview available in filebrowser
@frase-git
Copy link
Contributor Author

9e44be4 funktioniert!
Dachte gar nicht, dass das so einfach ist.

To clarify: this fix will only work as of PHP 7.0.10.

Wirft das dann bei kleineren PHP-Versionen einen Fehler?
Ansonsten hätte ich kein Problem damit, für XH allgemein PHP 7.2 vorauszusetzen.
Aber das wäre wohl ein zu großer Bruch - oder?

@frase-git
Copy link
Contributor Author

$plugin_cf['filebrowser']['extensions_images']="gif, jpg, jpeg, png, svg, webp";
... ist aber auch noch nötig.
Dann ist es wohl Sache der Dokumentation:
"Wenn du webp angezeigt bekommen möchtest, dann musst du min. PHP 7.0.10 haben und in der Config webp hinzufügen."

@cmb69
Copy link
Member

cmb69 commented Jun 2, 2021

Wirft das dann bei kleineren PHP-Versionen einen Fehler?

Nein, da wird einfach keine Vorschau angezeigt. Eigentlich nett, weil es User vielleicht zum Update bewegen wird. :) Und man muss ja nicht an die große Glocke hängen, dass die Vorschau bei PHP 5 ab PHP 5.6.25 auch funktioniert.

$plugin_cf['filebrowser']['extensions_images']="gif, jpg, jpeg, png, svg, webp";

Ich denke, dass sollten wir einfach noch machen. Bei Updates ist die Änderung irrelevant, und bei Neuinstallationen sehe ich kein Problem

@frase-git
Copy link
Contributor Author

Gut.

@cmb69
Copy link
Member

cmb69 commented Jun 2, 2021

Done; 8ff44d6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants