Salut Jacques,
il me semble que ce commit est malheureux.
Il casse le markup standard utilisé et que j'avais précisement uniformisé dans une série de commit il y a quelques semaines.
Je t'invite à regarder ce que cela donne si tu combine avec des champs extra sur une réponse 
Je suis favorable à un retour en arrière, quitte à styler en css 
Maïeul
Maïeul Rouquette a écrit le 24/05/2020 à 01:09 :
Author: Jacques Pyrat
Date: 2020-05-21 18:15:29 +0000 (Thu, 21 May 2020)
New Revision: 124776Modified:
_plugins_/formidable/trunk/
_plugins_/formidable/trunk/paquet.xml
_plugins_/formidable/trunk/prive/objets/contenu/formulaires_reponse.html
Log:
[Formidable] Densifier un peu l'affichage des reponses d'un formulaireDetails: Connexion · GitLab
Salut Jacques,
il me semble que ce commit est malheureux.
Il casse le markup standard utilisé et que j'avais précisement uniformisé dans une série de commit il y a quelques semaines.Je t'invite à regarder ce que cela donne si tu combine avec des champs extra sur une réponse
Je suis favorable à un retour en arrière, quitte à styler en css
Je vais regarder ça.
Il est bien possible que ça ait un rapport avec :
define('_SAISIES_AFFICHAGE_COMPACT', 'oui');
Est-ce que tu peux m'envoyer un formulaire avec des champs extras ?
Parce que je ne vois pas à quoi ça correspond.
RealET a écrit le 24/05/2020 à 09:58 :
Est-ce que tu peux m'envoyer un formulaire avec des champs extras ?
Parce que je ne vois pas à quoi ça correspond.
Je viens de créer un champ extra de test pour les formulaires.
Dans
ecrire/?exec=formulaire_edit&id_formulaire=1&configurer=formulaire
Ça me rajoute bien ce champ, que j'ai pu remplir.
Mais je ne le vois nul part ailleurs (ni dans le public, ni dans le privé)
Carrément : c’est quoi cette idée de se cuisiner son petit markup qui t’arrange bien sans tenir compte de la convention ?
Pourquoi ne pas simplement utiliser les css pour adapter à ton usage/besoin ?
display:inline ça sert à quoi ?
Et en plus on vire une class au passage, tranquille ?
Définitivement à revert !
des champs extra sur une réponse, pas sur un formulaire.
Exemple d'usage : des admin qui veulent ajouter des commentaires relatif à une réponse.
L'option _SAISIES_AFFICHAGE_COMPACT est je trouve une mauvaise idée aussi, en tout cas dans dans son implémentation actuelle.
Le css c'est pas fait pour rien.
Maïeul Rouquette a écrit le 24/05/2020 à 11:47 :
Cerdic a écrit le 24/05/2020 à 11:30 :
Carrément : c’est quoi cette idée de se cuisiner son petit markup qui t’arrange bien sans tenir compte de la convention ?
Pourquoi ne pas simplement utiliser les css pour adapter à ton usage/besoin ?
display:inline ça sert à quoi ?
Et en plus on vire une class au passage, tranquille ?Définitivement à revert !
Je fais ça dans la journée.
c'est fait entre deux debugs d'ibnstall chez moi.
a oui oki, pour ce cas.
quoi qu'il en soit, si tu veux avoir un affichage côté /ecrire qui soit cohérent avec la constante, il faut a minima la controler....
a mon sens là ca se ferait en css
Maïeul Rouquette a écrit le 24/05/2020 à 12:11 :
Cerdic a écrit le 24/05/2020 à 11:30 :
Carrément : c’est quoi cette idée de se cuisiner son petit markup qui t’arrange bien sans tenir compte de la convention ?
Pourquoi ne pas simplement utiliser les css pour adapter à ton usage/besoin ?
display:inline ça sert à quoi ?
Et en plus on vire une class au passage, tranquille ?Définitivement à revert !
Je fais ça dans la journée.
c'est fait entre deux debugs d'ibnstall chez moi.
Merci Maïeul 
Pour info, avec le revert (donc l'ancien affichage) et le define compact et un champ extra de réponse, voici une copie d'écran :
https://framapic.org/DqKpfjkG4dw0/VZ1Q6bMxMQiq.png
Qui paradoxalement affiche champ extra dans la version compacte des saisies (assez logique), et qui jure donc avec ce qui le précède.
Je vais voir comment faire ça en CSS.
Et voir si la CSS peut être conditionnée par le define.
Maïeul Rouquette a écrit le 24/05/2020 à 12:11 :
