Produits Entr'ouvert » w.c.s.

Autre idée (demande de ce matin à Alfortville)

• replier le résumé et le journal si la demande a déjà commencé à être traitée ;

• au niveau du journal, en mode replié, afficher quand même le dernier statut.

Il faut cependant savoir comment repérer que "la demande a déjà commencé à être traitée".

Ça me pose problème, ou en tant cas question, d'avoir ce détournement d'éléments graphiques. ("il y a du contenu sous le titre c'est donc que c'est déjà déplié, mais pourquoi la flèche est pas dans le bon sens ? et maintenant je veux replier oups non il y a plein de trucs qui apparaissent désormais").

Frédéric Péters a écrit :

Ça me pose problème, ou en tant cas question, d'avoir ce détournement d'éléments graphiques. ("il y a du contenu sous le titre c'est donc que c'est déjà déplié, mais pourquoi la flèche est pas dans le bon sens ? et maintenant je veux replier oups non il y a plein de trucs qui apparaissent désormais").

Tout à fait d'accord que ça marche pas (et c'est bien pour ça que j'ai posé ici ce que j'ai réussi à faire, pour discuter du soucis).

Il faut arriver à faire comprendre qu'il y a des éléments non affichés. Dans ma tête j'avais l'idée de faire apparaître au dessus du dernier élément un peu de l'avant dernier, genre en fondu dégradé... Et oui, la flèche ne colle pas.

Je n'ai pas d'exemple de site où ce principe existe, faudrait chercher un peu.

Pour clarifier, si on remplace les flèches par plier/déplier #9761, je trouve ça bien suffisant. Allé je vais faire un patch et un screencast.

À l'usage : sur le journal, ça ne va pas, parfois il faudrait afficher les 2 derniers status, et on comprend pas qu'il faut plier/déplier.

Je propose donc de ne garder seulement le pliage du résumé (déjà discuté avec Brice et Victor).

Pour moi c'est bizarre d'avoir la classe "folded" également ajoutée côté frontoffice alors qu'elle n'est pas prise en charge par les CSS; c'est quelque chose qu'on veut changer ?

Ouille ; je n'avais même pas perçu que c'était aussi la sortie frontoffice :/

En fait c'est la modif au _q_index qui me semble inutile parce que s'appliquant uniquement au frontoffice. (il ya un _q_index dans FormBackOfficeStatusPage, qui appelle self.status()).

Je trouve très très moche le :

r += htmltext('<h2 class="foldable%s">%%s</h2>' %
                      (' folded' if folded else '')) % _('Summary')

Je préférerais déjà :

        r += htmltext('<h2 class="foldable%s">' % (' folded' if deja_vu else ''))
        r += htmltext('%s</h2>') % _('Summary')

Voire même (parce que le if/else j'aime pas trop non plus) :

        klasses = 'foldable'
        if deja_vu:
            klasses += ' folded'
        r += htmltext('<h2 class="%s">' % klasses)
        r += htmltext('%s</h2>') % _('Summary')

(codes pas testé)

Dans les tests, pas d'espaces avant les deux-points (dans les commentaires).

Avec cette modif, ok.

Et en fait, les tests, ils m'effraient un peu à parler de "formdata is mine" et à faire ça alors que l'idée et le code concerne la présence de l'agent dans le formdata.evolution.

ok.