Bug #31700
Champ hidden pas caché
0%
Description
De #31697, un champ liste avec en extra css un hidden, mais qui se prend un style="display: block;" en plus.
Si j'en croit mon debuger ce serait qommon.form.js:87 avec $widget.show();
qui viendrait faire cela.
Et ici mon appréciation de la normalité ou non de la situation s'arrête.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 5 ans
- Projet changé de w.c.s. à Intégrations graphiques Publik
Ce qui se passe c'est qu'il y a calcul de la visibilité des champs parce que champs conditionnels dans la page et le calcul détermine que le champ doit être visible. (il est juste caché par un hack dans la CSS, hors scope w.c.s.).
Pour que rien ne vienne perturber la règle CSS le mot-clé est !important.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Fichier 0001-scss-force-hidden-class-to-take-over-everything-3170.patch 0001-scss-force-hidden-class-to-take-over-everything-3170.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ce patch reste une condamnation de la pratique.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
Testé sur le formulaire de Toulouse, c'est tout à fait étrange, dans le debugger le display: none !important;
prend le pas sur display: block;
qui se retrouve barré, mais le champ reste visible quand même.
Mis à jour par Frédéric Péters il y a environ 5 ans
Ça se teste avec un vrai fichier CSS, pas via l'inspecteur.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
J'ai testé avec patch appliqué et thèmes compilés, je ne fais que regarder ce qu'il se passe dans l'inspecteur sans y changer de propriété css.
Mis à jour par Frédéric Péters il y a environ 5 ans
Sur toulouse-test-1, j'ai pris /usr/share/publik/themes/publik-base/static/toulouse/style.css je l'ai copié dans /var/lib/combo/tenants/montoulouse.cutm-publik-preprod.nfrance.com/static/toulouse et j'y ai fait la modification (ligne 2159, ajout de !important). Et je constate que ça corrige bien l'affaire.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Ok alors go, je ne sais pas ce qui se passe chez moi.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e933ffe7b3a51d76a3b50fcd88a1f5d3000aba8c Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Mar 25 17:45:23 2019 +0100 scss: force hidden class to take over everything (#31700)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: force hidden class to take over everything (#31700)