Les dimensions des images ne sont plus prises en compte

Bug #1773540 reported by Aldian
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Documentation Ubuntu-fr
Invalid
Undecided
Unassigned

Bug Description

Bug observé sur https://doc.ubuntu-fr.org/wordpress

L'url des images possède un paramètre GET "?0x200". Par exemple:
https://codex.wordpress.org/images/e/e1/twentyelevendefault1.png?0x200

D'après la personne qui a remonté le problème, sur l'ancien thème, cela impliquait un redimensionnement automatique de l'image sur une hauteur de 200px, avec conservation des proportions en largeur. L'ensemble des captures d'écran s'affichait donc sur une seule ligne, ce qui n'est plus le cas, ils prennent désormais toute la largeur de la page.

Elle précise de plus que: "Une rapide inspection montre que le problème vient d'une règle height: auto; sur la classe .img-responsive de boostrap. Il faudrait sûrement retirer cette classe des images du contenu."

Revision history for this message
Aldian (aldian-gp) wrote :

Personnellement je n'étais pas au courant qu'on pouvait spécifier la taille des images de cette manière, et je ne suis pas certain que ce soit une bonne chose de l'encourager. Surtout dans la mesure où la vraie bonne pratique consiste à héberger sur le wiki les images du wiki au lieu de faire appel aux serveurs extérieurs.

Par contre on pourrait envisager de faire une recherche sur l'ensemble des pages qui possèdent ce paramètrage pour voir si cela représente un nombre important d'entre elles et en fonction du résultat, envisager de les corriger à la main en hébergeant une image de la bonne résolution, ou réfléchir à une solution alternative ne visant pas nécessairement à reproduire l'ancien comportement

En effet remettre en place l'ancien comportement implique de facto de renoncer à être responsive dans ce cas de figure.

Revision history for this message
kro (stiolbe) wrote :

merci aldian,

bonne chose ou pas c'est en tout cas une méthode que j'ai trouvée sur la doc officielle de dokuwiki :
https://www.dokuwiki.org/images
Or to scale an image proportionally in height, give the wanted height in Pixels, preceeded by a width of 0:

donc je pense que la bonne chose serait surtout que le thème respecte ce fonctionnement. d'autant que c'est la méthode la plus simple pour afficher une bande d'images ajustées en hauteur.

Revision history for this message
kro (stiolbe) wrote :

suite à conversation sur framateam, on peut fermer puisque le bug peut être résolu en uploadant les images localement...

il reste quand même quelques points d'ombre (mais il s'agirait plus d'un bug de fetch.php qu'un problème de CSS)... je copie colle mon monologue ici pour la postérité :

bon j'ai corrigé l'affichage des images sur la page wordpress en uploadant les images en local, et je m'apprétait à en faire de même sur la page grav quand je me suis apperçu que sur cette page ça fonctionne exactement comme je l'ai dit :

les images sont chargées depuis un lien externe mais des vignettes sont générées localement. dont voici un lien exact par ex. : https://doc.ubuntu-fr.org/lib/exe/fetch.php?h=200&tok=6dff1a&media=https%3A%2F%2Fgetgrav-grav.netdna-ssl.com%2Fuser%2Fpages%2F03.blog%2Fbeta-admin-plugin-available%2F00-login.png

honnêtement je n'y comprends plus rien, la syntaxe est exactement la même dans les deux cas :
https://doc.ubuntu-fr.org/grav
https://doc.ubuntu-fr.org/wordpress?rev=1524918084

Revision history for this message
Fabux (fabux) wrote :

ça pourrait provenir d'un "zero width space" .

J'ai copié/collé ce lien depuis la révision donnée
https://codex.wordpress.org/images/e/e1/twentyelevendefault1.png

après validation il s'est transformé en
https://codex.wordpress.org/%E2%80%8Bimages/%E2%80%8Be/%E2%80%8Be1/%E2%80%8Btwentyelevendefault1.png

Fabux (fabux)
Changed in ubuntu-fr-doc:
status: New → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.