No Branch/Tag Specified
1.8
1.9.1
1.9.2
2.0
2.1
3.0
3.1
3.2
4.0
4.1
boutons-danger
debug_ecrire_fichier
dev-sortable
dev/autoloader
dev/instituer_ergo
dev_infos_image
fix/valider_url_distante
fix_modifier_login
issue_4101
issue_4678
issue_4705
issue_4717
issue_4946
issue_5016
issue_5056_composer_road
issue_5273
master
refactor_texte_safety
v1.8.3+b
v1.9.1+i
v1.9.2+f
v1.9.2+g
v1.9.2+h
v1.9.2+i
v1.9.2+j
v1.9.2+k
v1.9.2+m
v1.9.2+n
v1.9.2+o
v1.9.2+p
v2.0.0
v2.0.1
v2.0.10
v2.0.11
v2.0.12
v2.0.13
v2.0.14
v2.0.15
v2.0.16
v2.0.17
v2.0.18
v2.0.19
v2.0.2
v2.0.20
v2.0.21
v2.0.22
v2.0.23
v2.0.24
v2.0.25
v2.0.26
v2.0.3
v2.0.5
v2.0.6
v2.0.7
v2.0.8
v2.0.9
v2.1.0
v2.1.1
v2.1.10
v2.1.11
v2.1.12
v2.1.13
v2.1.14
v2.1.15
v2.1.16
v2.1.17
v2.1.18
v2.1.19
v2.1.2
v2.1.20
v2.1.21
v2.1.22
v2.1.23
v2.1.24
v2.1.25
v2.1.26
v2.1.27
v2.1.28
v2.1.29
v2.1.3
v2.1.30
v2.1.4
v2.1.5
v2.1.6
v2.1.7
v2.1.8
v2.1.9
v3.0.0
v3.0.0-alpha.1
v3.0.0-beta
v3.0.0-beta.2
v3.0.0-rc
v3.0.1
v3.0.10
v3.0.11
v3.0.12
v3.0.13
v3.0.14
v3.0.15
v3.0.16
v3.0.17
v3.0.18
v3.0.19
v3.0.2
v3.0.20
v3.0.21
v3.0.22
v3.0.23
v3.0.24
v3.0.25
v3.0.26
v3.0.27
v3.0.28
v3.0.3
v3.0.4
v3.0.5
v3.0.6
v3.0.7
v3.0.8
v3.0.9
v3.1.0
v3.1.0-alpha
v3.1.0-beta
v3.1.0-rc
v3.1.0-rc.2
v3.1.0-rc.3
v3.1.1
v3.1.10
v3.1.11
v3.1.12
v3.1.13
v3.1.14
v3.1.15
v3.1.2
v3.1.3
v3.1.4
v3.1.5
v3.1.6
v3.1.7
v3.1.8
v3.1.9
v3.2-alpha.1
v3.2.0
v3.2.0-alpha.1
v3.2.0-beta
v3.2.0-beta.2
v3.2.0-beta.3
v3.2.1
v3.2.10
v3.2.11
v3.2.12
v3.2.13
v3.2.14
v3.2.15
v3.2.16
v3.2.2
v3.2.3
v3.2.4
v3.2.5
v3.2.6
v3.2.7
v3.2.8
v3.2.9
v4.0.0
v4.0.0-alpha
v4.0.0-beta
v4.0.1
v4.0.2
v4.0.3
v4.0.4
v4.0.5
v4.0.6
v4.0.7
v4.0.8
v4.1.0
v4.1.0-alpha
v4.1.0-beta
v4.1.0-rc
v4.1.1
v4.1.2
v4.1.3
v4.1.4
v4.1.5
Labels
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
Apply labels
Clear labels
accessibilité
amélioration
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
No Label
accessibilité
amélioration
APIs
authentification
base de données
bug
code généré
compilo
css
divers
documentation
doublon
ergonomie
espace privé
filtres et balises
formulaires
Inscription
installation
invalide
javascript
langues
LDAP
plugin
PostgreSQL
refusé
sécurité
traduction
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
This issue currently doesn't have any dependencies.
Reference in new issue
There is no content yet.
Delete Branch '%!s(MISSING)'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
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.Ç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:
, 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).
Assigné à cedric
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 biengif,jpg,png,webp
, chelou ?Statut changé à En cours
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 ?
Cf spip/medias#4815
En fait quand on sait pas réduire une image parce que format pas supporté comme ici, le fallback c'est de garder l'image d'origine et de la réduire juste visuellement via un max-height en dur sur la balise img
Ici ça donne
du coup le
max-width
en inline prend la main sur celui de la css qui était unmax-width:100%
On devrait donc plutôt insérer un
max-width: min(100%,320px);
pour être plus robuste.Mais en regardant le code, on voit que ce max-width est un fallback du fallback, c'est à dire si on connait pas du tout les dimensions du fichier parce qu'il n'y a ni width/height sur la balise img.
On aurait pas eu le bug si au départ on avait appliqué un filtre
|balise_img
avant d'appliquer|image_reduire
.Mais on peut facilement améliorer en appelant
taille_image()
qui sera plus robuste.Je propose donc un double patch :
Testé et ça marche : le rendu du logo dans sa colonne est parfait.
Au passage bravo SPIP et Cerdic pour cette épatante simulation des traitements d'images lorsque la g-librairie ne les prend pas en charge.
On ferme.