Logos webp débordent #4826

Open
opened 3 months ago by JLuc · 4 comments
JLuc commented 3 months ago

Les logos débordent de leur colonne sur l'espace du texte principal lorsqu'ils sont trop grands. Constaté sur contrib, allwaysdata mutu, OVH mutu et gandi simplehosting. Sur ces 2 derniers au moints, le "WebP Support" est "enabled".
Page exemple où ça se voit : https://contrib.spip.net/ecrire/?exec=article&id_article=5288

Le pb semble être que c'est le fichier original du logo qui est affiché, redimensionné par le html (width=, comme les logos dans les listes d'ailleurs) à une largeur de 320px calculée et imposée par SPIP : img src="../IMG/logo/framboise.resized.webp?1623097946" style="max-width: 320px; max-height: 320px" alt="logo_on"
Mais la colonne ne fait que 286px de large d'après l'inspecteur.

Ça se joue dans prive/formulaires.inc-apercu-logo.html dont le dernier log est "arrêter les logos riquiqui" et passe la largeur de 300 à 320px : d5ae227fd4 . Mais 320 ça semble trop ou alors faut aussi élargir la colonne.

Les logos débordent de leur colonne sur l'espace du texte principal lorsqu'ils sont trop grands. Constaté sur contrib, allwaysdata mutu, OVH mutu et gandi simplehosting. Sur ces 2 derniers au moints, le "WebP Support" est "enabled". Page exemple où ça se voit : https://contrib.spip.net/ecrire/?exec=article&id_article=5288 Le pb semble être que c'est le fichier original du logo qui est affiché, redimensionné par le html (width=, comme les logos dans les listes d'ailleurs) à une largeur de 320px calculée et imposée par SPIP : img src="../IMG/logo/framboise.resized.webp?1623097946" style="max-width: 320px; max-height: 320px" alt="logo_on" Mais la colonne ne fait que 286px de large d'après l'inspecteur. Ça se joue dans [prive/formulaires.inc-apercu-logo.html](https://git.spip.net/spip/spip/src/branch/master/prive/formulaires/inc-apercu-logo.html) dont le dernier log est "arrêter les logos riquiqui" et passe la largeur de 300 à 320px : https://git.spip.net/spip/spip/commit/d5ae227fd47a94d1148f7d34b6f616147bf78816 . Mais 320 ça semble trop ou alors faut aussi élargir la colonne.
Collaborator

Ça n'a rien à voir avec la modification citée (parce que 300 pixels, ça dépassera aussi :-)).

Le souci c'est que le image_reduire ne s'applique pas sur une image WebP.

La raison étant que WebP n'est pas dans les formats déjà détectés et stockés dans:

$GLOBALS['meta']['gd_formats']
, et donc on échoue sur “_image_extensions_acceptees_en_sortie()” qui lui-même passe par “_image_extensions_acceptees_en_entree()” (c'est lui qui refuse WebP).

Il faudrait donc supprimer et recalculer la valeur du méta lors de l'activation de WebP (je ne sais pas faire).

Ça n'a rien à voir avec la modification citée (parce que 300 pixels, ça dépassera aussi :-)). Le souci c'est que le `image_reduire` ne s'applique pas sur une image WebP. La raison étant que WebP n'est pas dans les formats *déjà détectés* et stockés dans: <pre>$GLOBALS['meta']['gd_formats']</pre>, et donc on échoue sur “_image_extensions_acceptees_en_sortie()” qui lui-même passe par “_image_extensions_acceptees_en_entree()” (c'est lui qui refuse WebP). Il faudrait donc supprimer et recalculer la valeur du méta lors de l'activation de WebP (je ne sais pas faire).
Collaborator

Assigné à cedric

**Assigné à cedric**
Owner

Comme je le disais dans https://core.spip.net/issues/4756#note-22 "ça marche chez moi ©", et si je dump $GLOBALS['meta']['gd_formats'] j'ai bien gif,jpg,png,webp, chelou ?
Statut changé à En cours

Comme je le disais dans https://core.spip.net/issues/4756#note-22 "ça marche chez moi ©", et si je dump `$GLOBALS['meta']['gd_formats']` j'ai bien `gif,jpg,png,webp`, chelou ? **Statut changé à En cours**
Poster

Modifier la valeur de l'option "Autoriser la génération de vignette" a un impact sur ce problème. Quand on autorise la génération des vignettes, il semble que la vignette ne déborde pas.
Attention : Il y a un hystérésis ensuite puisqu'une fois la vignette créée elle est utilisée même si on retire l'autorisation de génération. Ça rend le test plus délicat.

Le problème serait il dû au code HTML généré lorsque image_reduire n'est pas appelé ?

Au passage je vois que la fonction action_tester_dist à la fois fait un test de transformation d'image ET écrit la meta gd_format. N'est ce pas une confusion des genres ?

Modifier la valeur de l'option "Autoriser la génération de vignette" a un impact sur ce problème. Quand on autorise la génération des vignettes, il semble que la vignette ne déborde pas. Attention : Il y a un hystérésis ensuite puisqu'une fois la vignette créée elle est utilisée même si on retire l'autorisation de génération. Ça rend le test plus délicat. Le problème serait il dû au code HTML généré lorsque image_reduire n'est pas appelé ? Au passage je vois que la fonction action_tester_dist à la fois fait un test de transformation d'image ET écrit la meta gd_format. N'est ce pas une confusion des genres ?
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.