Projet

Général

Profil

Bug #5094

résumé d'une demande : Affichage des textarea dans les dataview

Ajouté par Victor Claudet il y a presque 10 ans. Mis à jour il y a plus de 9 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 juillet 2014
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Un bug graphique un peu bizarre pour l'affichage des textarea dans les dataview.

Le code généré par au quotidien est le suivant :

<p>
<span class="label">Adresse</span> 
<span class="value">
<p>le texte saisi par l'usager</p>
</span>
</p>

Le code exécuté par le navigateur :

<p>
<span class="label">Adresse</span> 
<span class="value"></span>
</p>
<p>le texte saisi par l'usager</p>
<p></p>

Du coup mes styles sont pas pris en compte pour cet élément


Fichiers

Historique

#2

Mis à jour par Frédéric Péters il y a presque 10 ans

J'ai modifié le markup mais ça va signifier inspection de tous les thèmes installés partout :/

Principalement un <p> remplacé par un <div>, ce qui nécessite donc d'être explicite sur les marges :

+div.dataview div.field {
+       margin: 1em 0;
+}

Et le span.value remplacé par un div,

-div.dataview span.value {
+div.dataview .value {
        display: block;
        margin-left: 1em;
 }

(il peut rester des span.value à certains endroits, j'ai donc préféré ça à un remplacement par div.value).

#3

Mis à jour par Thomas Noël il y a presque 10 ans

  • Statut changé de Nouveau à En cours

Ok pour le patch en lui-même.

Reste à voir si c'est ok pour Victor (i.e. revoir les css des autres sites).

#4

Mis à jour par Frédéric Péters il y a presque 10 ans

J'ai appliqué la modif de CSS sur les sites déployés;

sur auquo.entrouvert.org :

collectivites-cam.au-quotidien.com/themes/montpelliercollectivites/wcs.css
demo-beaumont.au-quotidien.com/themes/beaumont/wcs.css
demo-creil.au-quotidien.com/themes/creil/wcs.css
demo-echirolles.au-quotidien.com/themes/auquotidien-echirolles/wcs.css
demo-gonfreville.au-quotidien.com/themes/auquotidien-gonfreville/wcs.css
demo-lambersart.au-quotidien.com/themes/lambersart2/wcs.css
demo-lambersart.au-quotidien.com/themes/lambersart/wcs.css
demo-marcoussis.au-quotidien.com/themes/auquotidien-marcoussis/wcs.css
demo-ng.au-quotidien.com/themes/auquotidien-noyelles-godault/wcs.css
echirolles.au-quotidien.com/themes/auquotidien-echirolles/wcs.css
gonfreville-l-orcher.au-quotidien.com/themes/auquotidien-gonfreville/wcs.css
lambersart.au-quotidien.com/themes/lambersart2/wcs.css
lambersart.au-quotidien.com/themes/lambersart/wcs.css
marcoussis.au-quotidien.com/themes/auquotidien-marcoussis/wcs.css
mon-villeneuve-demo.au-quotidien.com/themes/villeneuvelagarenne/wcs.css
noyelles-godault.au-quotidien.com/themes/auquotidien-noyelles-godault/wcs.css
vandoeuvre.au-quotidien.com/themes/auquotidien-vandoeuvre/wcs.css
victor.au-quotidien.com/themes/auquo2/wcs.css
victor.au-quotidien.com/themes/aurillac/wcs.css
victor.au-quotidien.com/themes/calvados3/wcs.css
victor.au-quotidien.com/themes/creil/wcs.css
victor.au-quotidien.com/themes/cud/wcs.css
victor.au-quotidien.com/themes/meyzieu/wcs.css
victor.au-quotidien.com/themes/montpellier/wcs.css
victor.au-quotidien.com/themes/villeneuvelagarenne/wcs.css

et sur auquo-test.entrouvert.org :

cud.au-quotidien.com/themes/cud/wcs.css
demo-calvados.test.au-quotidien.com/themes/calvados2/wcs.css
echirolles.dev.au-quotidien.com/themes/auquotidien-echirolles/wcs.css
e-megalis.test.au-quotidien.com/themes/auquotidien-echirolles/wcs.css
e-megalis.test.au-quotidien.com/themes/aurillac/wcs.css
e-megalis.test.au-quotidien.com/themes/megalis/wcs.css
ng.dev.au-quotidien.com/themes/auquotidien-noyelles-godault/wcs.css
paiement.test.au-quotidien.com/themes/iframe/wcs.css
poissy.test.au-quotidien.com/themes/poissy/wcs.css
test-vincennes.au-quotidien.com/themes/auquotidien-vincennes/wcs.css
test-vincennes.au-quotidien.com/themes/aurillac/wcs.css
test-vincennes.au-quotidien.com/themes/theme/wcs.css

J'ai aussi un patch pour auquotidien (attaché) et un autre pour montpellier-themes-wcs (déjà poussé).

#5

Mis à jour par Thomas Noël il y a presque 10 ans

Merci Fred, tu peux pousser les patches, alors.

#6

Mis à jour par Frédéric Péters il y a presque 10 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 2621a712a1e0de0283c1fe69b69b2583902a4391
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Jul 4 09:17:20 2014 +0200

    use a <div> to enclose field values, as they can be block elements (#5094)
#7

Mis à jour par Frédéric Péters il y a presque 10 ans

C'est poussé et j'ai adapté la css de Vincennes (vive bootstrap qui faisait un .label { color: white; }).

#8

Mis à jour par Thomas Noël il y a plus de 9 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Thomas Noël il y a plus de 9 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF