Projet

Général

Profil

Development #18779

backoffice : ajouter les champs "titres" dans le résumé d'une demande

Ajouté par Thomas Noël il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
19 septembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour améliorer la lisibilité, c'est sans doute pertinent.


Fichiers

Révisions associées

Révision f6e89c2a (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

forms: include titles and subtitles in summary page (#18779)

Historique

#2

Mis à jour par Frédéric Péters il y a plus de 6 ans

Et les sous-titres ? Et les villes qui tapent des titres dans les champs commentaire ?

#3

Mis à jour par Pierre Cros il y a plus de 6 ans

De mon point de vue à chaque jour suffit sa peine. J'ignorerais les sous-titres et les commentaires.

Et il faudra indiquer aux villes qui mettraient des titres dans les commentaires et qui s'étonneraient de ne pas les retrouver dans l'interface de traitement, d'utiliser le champ adéquat pour ça.

#4

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

Pour moi, reprendre ce qui est affiché dans la page de validation (titres, mais pas les commentaires)

#5

Mis à jour par Pierre Cros il y a environ 6 ans

Concernant les sous-titres, Sébastien préfère qu'on les inclue et moi aussi du coup.

#10

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

J'ai repris quand même les sous-titres parce que le formulaire pointé en #24115 en comportait.

#11

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

Juste un détail, parce que je comparais page de validation et résumé (puisqu'ici on les rend identiques, ce que j'aime bien) : dans la page de validation, on utilise h3 pour les titres et h4 pour les sous-titres. Peut-être un rapprochement utile pour la gestion des css et des thèmes ?

Sinon, ack.

#12

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

J'ai hésité, en front sur la page de validation on a <h3> aussi bien pour les pages que les titres, ça ne me semblait pas à répliquer (mais ça se discute, aucune urgence ici, je vais dormir dessus).

#13

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

Bonne nuit. Tu te couches drôlement tôt, je trouve.

#14

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

Un peu plus chiant à styler; aussi, pour une bonne gestion de l'alignement avec les classes de grille, ajout de <div> comme aux autres widgets (sauf sur la page de validation, etc. mais ce n'est pas dans ce ticket qu'on synchronisera pile le balisage entre ces vues).

#15

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

Finalement c'est pas plus simple, je pensais qu'on pourrait se passer d'un patch sur le scss... mais vu qu'en css on aime bien répéter 10 fois un truc, peut-être que non.

Tu choisis ce qui te sembles le plus agréable ("Un peu plus chiant à styler"), l'idée est la même dans les deux patches et je acke "les deux".

#16

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

  • Statut changé de En cours à Résolu (à déployer)

Pas de préférence nette même après avoir pas mal dormi dessus, j'ai poussé la dernière version.

commit f6e89c2a250a7e4c6b770ae2f15b88463e81423c
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon May 28 12:53:30 2018 +0200

    forms: include titles and subtitles in summary page (#18779)
#17

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Formats disponibles : Atom PDF