Development #83018
les class grid-*-* et newline ne fonctionnent pas (ou pas comme elles devraient) sur les widgets titre et sous-titre
0%
Description
cf. https://dev.entrouvert.org/issues/81925
si le comportement normal attendu et que cela fonctionne, il faudrait corriger ça.
Thomas Jund a écrit :
J'ai regardé s'il était envisageable d'ajouter le fonctionnement de la grille aux widgets titre et sous-titre.
Après enquête mené par Pierre D, il a en prod 2615 widgets titre et sous-titre avec la class newline, et 3186 avec une class grid-*-*.
Ce qui veut dire que si j'ajoute le float et le clear là dessus, ça risque de bien bouger.
(Je crois que je n'ai pas de solution, ou alors on y va en annonçant que ça peut bouger)
ce sera à la marge, j'ai pas l'impression que beaucoup de titre et sous-titre ont un grid de posé, par contre un newline, a priori oui puisqu'on dit lors des formations à nos clients que s'ils en mettent sur au moins un champ, ils doivent en mettre partout.
Je pense qu'il faut corriger ça, sauf si je loupe quelque chose,
Historique
Mis à jour par Frédéric Péters il y a 6 mois
- Projet changé de w.c.s. à Intégrations graphiques Publik
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a 6 mois
ce sera à la marge, j'ai pas l'impression que beaucoup de titre et sous-titre ont un grid de posé, par contre un newline, a priori oui puisqu'on dit lors des formations à nos clients que s'ils en mettent sur au moins un champ, ils doivent en mettre partout.
Comme dit dans mon commentaire, ça peut bouger sur tous les widgets qui ont déjà les class grid et newline.
Très difficile pour moi à évaluer si ce sera "à la marge" sur les 3000 widgets concernés (combien font partis de demandes publiés ?).
Comme les demandes en prod sont à mon avis très peu retestées par les collectivités, on ne saura pas ce qui a bougé ou pas.
Mais je peux proposer un patch.
Mis à jour par Anaïs Ecuvillon → en congés, retour le 30/04 il y a 6 mois
Thomas Jund a écrit :
Comme dit dans mon commentaire, ça peut bouger sur tous les widgets qui ont déjà les class grid et newline.
Concrètement, ça peut bouger, ça veut dire quoi ? Je ne me rends pas compte de l'impact, donc impossible pour moi de prendre une décision éclairée,
Mis à jour par Frédéric Péters il y a 6 mois
En y allant plus doucement,
- 1/ je ne reproduis pas de problème dans la démarche évoquée dans le ticket lié,
- 2/ on y parle de titres dans des blocs de champs, information qui n'est pas reprise dans ce ticket,
- 3/ la logique attendue pour les titres est qu'ils forcent le retour à la ligne,
form div[class*=grid-] {
~ h3, ~ h4, ~ p, + div {
clear: both;
}
- 4/ on comprend alors l'importance de préciser qu'il s'agit de blocs de champs, où la balise du titre va arriver en premier, sans suivre un div.grid-
Bref je serais pour d'abord vérifier qu'il y a bien toujours un problème (spoiler : oui, si les blocs n'ont pas de bouton de suppression), et moins le généraliser.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a 6 mois
la logique attendue pour les titres est qu'ils forcent le retour à la ligne,
Dans ce cas, je me dis qu'on pourrait forcer cela pour de vrai, systématiquement, et pas uniquement "si le titre suit à un moment donnée un champ positionné en grille".
Mis à jour par Frédéric Péters il y a 6 mois
Mais tu te mets l'obstacle de vérifier partout front et back tous les formulaires, ça serait plus sûr de simplement viser les (sous-)titres des blocs de champs.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a 6 mois
ça serait plus sûr de simplement viser les (sous-)titres des blocs de champs
Ok, je fais ça.