[spip-dev] SPIP 3.3 et Compositions

Dev
1

Salut,

sur une installation locale avec SPIP-Cli (3.3.0-dev GIT [master: cc4985ef]), le nouvel icône SVG cadenas (compositions verrouillées) s'affiche mal, que les compostions soient dépliées ou pas : https://pic.infini.fr/ITLK84Ru/Xac3w64x.png

Aussi, depuis quelques temps, j'ai un bug d'affichage étrange en 3.3 avec Compositions (3.8.0) : dans un article lorsque je déplie les compositions pour en changer, le formulaire s'affiche mal : https://pic.infini.fr/172bphJy/E3lHKJtQ.png
Si j'ajoute un overflow: auto; sur .formulaire_spip .choix, ça corrige le problème.

Le bug est étrange car je n'arrive pas à reproduire sur tous mes sites... Est-ce que quelqu'un a constaté ça aussi ?

             jeanmarie
2

Hop,

Salut,

sur une installation locale avec SPIP-Cli (3.3.0-dev GIT [master: cc4985ef]), le nouvel icône SVG cadenas (compositions verrouillées) s'affiche mal, que les compostions soient dépliées ou pas : https://pic.infini.fr/ITLK84Ru/Xac3w64x.png

Je reproduis ce bug.

Aussi, depuis quelques temps, j'ai un bug d'affichage étrange en 3.3 avec Compositions (3.8.0) : dans un article lorsque je déplie les compositions pour en changer, le formulaire s'affiche mal : https://pic.infini.fr/172bphJy/E3lHKJtQ.png
Si j'ajoute un overflow: auto; sur .formulaire_spip .choix, ça corrige le problème.

Mais pas celui-ci...

3

Re,

4

Hop,

5

Pour info, par quelle sorcellerie est-ce que <img src="#CHEMIN_IMAGE{cadenas-16.png}" />
va chercher prive/themes/spip/images/cadenas-xx.svg ?

JL

6

Ça me semble très bien que chemin_image() privilégie la version svg si elle est présente (si c'est fait àce niveau, j'ai pas regardé).

7

Point de sorcellerie, juste de la magie :slight_smile:

https://git.spip.net/spip/spip/commit/b572418d55c7cad0d9e1542a9f7f18e8ff7db25e

8

Hihi non je ne crois pas que cette ligne spoile le "truc"
car elle teste si le fichier existe, auquel cas ça le renvoie,
et donc ok ça permet à #CHEMIN_IMAGE de renvoyer un SVG reçu en paramètre...
mais dans le code de composition corrigé, #CHEMIN_IMAGE reçoit un PNG en argument
<img width="16" height="16" src="#CHEMIN_IMAGE{cadenas-16.png}" alt="" />
et renvoie un SVG
c'est bien ça ?
Alors c'est autre chose. Ça pourrait être dans
https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/icone_renommer.php
qui fait de la magie aussi, mais je vois pas où car nulle par ça ne favorise un svg
et le fichier n'a pas changé dans la branche "icones_svg"
...

JL

9

Hello,

C’est là
https://git.spip.net/spip/spip/commit/9ef4c078fae4ccef00782a3901b340fe9888d2d3

10

Et aussi un peu par ici si je ne me trompe pas :slight_smile:

https://git.spip.net/spip/spip/commit/da298555d81f323361ca320fb8d3c97307390742

11

Bruno Bergot a écrit Et aussi un peu par ici si je ne me trompe pas :)> https://git.spip.net/spip/spip/commit/da298555d81f323361ca320fb8d3c97307390742
Ça le fait effectivement ici pour les icones et les puces,
mais http_img_pack n'est pas appelé dans le code précité de composition
qui ne contient qu'un pur <img src=#CHEMIN_IMAGE>

Par contre je crois avoir trouvé : ya le même code dans find_in_theme
qui lui est bien appelé par chemin_image :
https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/utils.php#L1427

OK
JLuc

12

Hop,

13

C’est le bon usage oui, mais...

il nous manque la possibilité de spécifier la taille que l’on veut sur le filtre balise_img car le filtre a été fait a une epoque sans retina, et en vrai aujourd’hui on voudrait plutôt faire un truc du genre
[(#CHEMIN_IMAGE{cadenas-32.png}|balise_img{16})]
ou
[(#CHEMIN_IMAGE{cadenas-xx.svg}|balise_img{16})]

j’ai donc filtre plus complet en chantier que j’essaye de stabiliser et je reflechi comment gérer la transition en cassant le moins de chose, notamment je pense qu’il faut supporter des syntaxes du genre :

// images carrees
[(#CHEMIN_IMAGE{cadenas-xx.svg}|balise_img{16})]

// specifier la resolution plutot que la taille explicite
[(#CHEMIN_IMAGE{cadenas-32.png}|balise_img{x2})]

// specifier largeurxhauteur
[(#CHEMIN_IMAGE{truc.svg}|balise_img{16x32})]

Je ne sais pas encore si on peut gérer la transition en étendant le filtre balise_img et sans casser ses usages anciens tout en supportant la nouvelle syntaxe, et sinon il faudra lui trouver un joli nom explicite