Bug #31744
CD13 : Ajouter dans la css les styles des cellules textcell et dataview
Début:
26 mars 2019
Echéance:
28 mars 2019
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Reprendre le style dans la css pour pouvoir virer :
<div class="textcell" style="padding: 1rem;"> <div class="dataview" style="display: block;">
Fichiers
Demandes liées
Révisions associées
bouches-du-rhone: add proper padding instead of abusing textcell class (#31744)
bouches-du-rhone: remove <br> abuse and invalid <p> markup (#31744)
bouches-du-rhone: add colour to custom cell subtitles (#31744)
bouches-du-rhone: use own class for displaying data fields (#31744)
bouches-du-rhone: remove inappropriate centering (#31744)
Historique
Mis à jour par Mikaël Ates il y a environ 5 ans
- Lié à Development #31726: CD13: intégration des cellules JSON au thème ajouté
Mis à jour par Frédéric Péters il y a environ 5 ans
- Tracker changé de Support à Bug
- Echéance changé de 05 avril 2019 à 28 mars 2019
- Assigné à changé de Emmanuel Cazenave à Frédéric Péters
Que je regarde pour corriger ça avant le passage en production, y a-t-il quelque part un exemple des données JSON attendues par les différentes cellules ?
Mis à jour par Frédéric Péters il y a environ 5 ans
- Fichier cd13_dossier_nom.json cd13_dossier_nom.json ajouté
- Fichier cd13_dossier_formulaires.json cd13_dossier_formulaires.json ajouté
- Fichier cd13_dossier_entete.json cd13_dossier_entete.json ajouté
- Fichier cd13_dossier_demandes.json cd13_dossier_demandes.json ajouté
- Fichier cd13_dossier_coordonnees.json cd13_dossier_coordonnees.json ajouté
- Fichier cd13_dossier_contacts.json cd13_dossier_contacts.json ajouté
Mis à jour par Frédéric Péters il y a environ 5 ans
- Fichier 0006-bouches-du-rhone-remove-inappropriate-centering-3174.patch 0006-bouches-du-rhone-remove-inappropriate-centering-3174.patch ajouté
- Fichier 0005-bouches-du-rhone-use-own-class-for-displaying-data-f.patch 0005-bouches-du-rhone-use-own-class-for-displaying-data-f.patch ajouté
- Fichier 0004-bouches-du-rhone-add-colour-to-custom-cell-subtitles.patch 0004-bouches-du-rhone-add-colour-to-custom-cell-subtitles.patch ajouté
- Fichier 0003-bouches-du-rhone-remove-br-abuse-and-invalid-p-marku.patch 0003-bouches-du-rhone-remove-br-abuse-and-invalid-p-marku.patch ajouté
- Fichier 0002-bouches-du-rhone-add-proper-padding-instead-of-abusi.patch 0002-bouches-du-rhone-add-proper-padding-instead-of-abusi.patch ajouté
- Fichier 0001-bouches-du-rhone-don-t-let-title-cover-cell-borders-.patch 0001-bouches-du-rhone-don-t-let-title-cover-cell-borders-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
- 0001 : pas directement lié mais il m'a semblé que l'ajout de .textcell pouvait venir du sale rendu des titres dont le fond blanc couvre les bordures; j'ai donc ajouté le flag adéquat pour retirer ce comportement (qui a du sens quand le titre a un fond de couleur).
- 0002 : suppression des textcell et ajout du padding nécessaire
- 0003 : suppression des <br> et de <p> mal placés (genre autour d'un <table>)
- 0004 : ajout de couleurs aux sous-titres (ils l'avaient perdues en 0002)
- 0005 : ajout d'un style dédié pour les alignements de champs libellés/valeurs
- 0006 : suppression de style="text-align: center" et remplacement par rien parce que je trouve ce centrage inutile.
Visuellement il y a donc ce 0006 qui change un peu les choses et il y a la cellule cd13_dossier_nom où je mets le titre avec le même padding qu'ailleurs
Il y aurait à nettoyer davantage le code, il y a parfois des doubles espaces mal placés,
{% if json.data.dossier.demandes %}
<h3 > Demandes</h3>
{% if json.data.dossier.demandes.en_cours %}
mais je zappe ça.
Autre chose qu'il y aurait à faire c'est utiliser les classes info et autres pour ajouter du style à des messages genre "Vous n'avez aucune traitée mais non encore expédiée.".
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Je dirai que c'est déjà une bonne base, merci.
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 28fa74ad290b31f77cdad182c4977da96680f7a9
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 26 21:07:27 2019 +0100
bouches-du-rhone: remove inappropriate centering (#31744)
commit a6e1fb9a2e2676c8559cb5a44b971097a487d9d1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 26 20:51:54 2019 +0100
bouches-du-rhone: use own class for displaying data fields (#31744)
commit 36bc8521b00b521b7ed261749033a3e0e5f9c505
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 26 20:39:07 2019 +0100
bouches-du-rhone: add colour to custom cell subtitles (#31744)
commit c8b1db23d791095c87cfb66266feea44f33ee9be
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 26 20:36:50 2019 +0100
bouches-du-rhone: remove <br> abuse and invalid <p> markup (#31744)
commit 2a080a20c7ea5b9975609923a605486b6945ab1c
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 26 20:33:40 2019 +0100
bouches-du-rhone: add proper padding instead of abusing textcell class (#31744)
commit c4c6316b8da2fa08fd33461b7de0e53bd5340f1d
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 26 20:25:06 2019 +0100
bouches-du-rhone: don't let title cover cell borders (#31744)
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
bouches-du-rhone: don't let title cover cell borders (#31744)