Development #27618
Pouvoir déplier / replier les sous-pages en backoffice
0%
Description
J'étais convaincu que ce ticket existait et je n'ai pas trouvé. C'est pour la doc Publik.
Le nombre de pages augmente, la lisibilité en backoffice diminue.
Il faudrait que les sous-pages soit masquées par défaut et pouvoir les afficher en cliquant sur une flèche (dirigée vers la droite) située à droite du libellé de la page mère.
Après clic, on voit les sous-pages, la flèche devient une flèche dirigée vers le bas, cliquable pour masquer à nouveau les sous-pages et revenir au statu quo ante.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 5 ans
- Sujet changé de Pourvoir déplier / replier les sous-pages en backoffice à Pouvoir déplier / replier les sous-pages en backoffice
Mis à jour par Frédéric Péters il y a environ 4 ans
- Dupliqué par Development #41467: Rendre "foldable" la liste des pages dans Combo ajouté
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0001-manager-toggle-pages-in-manager-home-27618.patch 0001-manager-toggle-pages-in-manager-home-27618.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Jund il y a environ 4 ans
Petit retour UI/UX,
La zone de clic est limitée uniquement au chevron, je la verrais plus large.
J'indiquerais un peu plus fortement la présence de ce nouvel élément "clickable" (qu'il serait exagéré d'appeler un bouton), avec un léger background color et/ou une taille plus importante (le style guide de gadjo est peut-être limité sur l'UI des boutons d'interface, j'ai pas regardé).
Indiquer le nombre de pages cachées serait peut-être une bonne idée ?
Ajout qui engendre une petite régression au niveau UX sur le déplacement des pages : On ne peut plus déplacer une page au niveau des sous-pages cachées. Il faut d’abord déplier les sous-pages pour ensuite pouvoir y déplacer une page. (C'est pas forcement une mauvaise chose).
Dev minimaliste qui fonctionne bien avec le code existant, mais au détriment de la sémantique/accessibilité, de l'UX et de la performance (le DOM est traversé à chaque fois qu'on ouvre/ferme). Mais peut-être que ça pourrait venir par des améliorations suivantes ?
Mis à jour par Pierre Cros il y a environ 4 ans
Thomas Jund a écrit :
peut-être que ça pourrait venir par des améliorations suivantes ?
Je milite fortement pour une ticket différent avec toutes ces intéressantes propositions d'amélioration.
Avoir le travail en l'état dans la release de dans 15 jours, ça me serait très utile pour actualiser la doc avant la fin du confinement (en reprenant toutes les mises à jour depuis 6 mois ou en an je sais plus trop).
Mis à jour par Lauréline Guérin il y a environ 4 ans
Alors personnellement je me suis bien cassée les dents sur ce bout de JS, j'arrive au bout de mes compétentes en front :)
Je vous propose soit:
- une personne plus expérimentée reprend le ticket (Thomas J ?)
- on intègre en l'état (modulo quelques corrections si nécessaire bien sûr), et on crée des tickets d'amélioration
Mis à jour par Thomas Jund il y a environ 4 ans
Je vous propose soit:
- une personne plus expérimentée reprend le ticket (Thomas J ?)
- on intègre en l'état (modulo quelques corrections si nécessaire bien sûr), et on crée des tickets d'amélioration
Le code fait le job demandé. Dans l'état il peut être poussé. Pierre en a besoin pour la prochaine release. Pas de problème pour moi.
Ma crainte principale est du coté des clients qui ne vont plus voir leurs pages et qui ne vont pas voir le chevron qui est assez discret. Mais côté BO c'est pas blocant du tout.
Je vais voir si je peux modifier un peu les CSS.
Sinon ça passera en l'état.
Mis à jour par Pierre Cros il y a environ 4 ans
Juste parce que je trouve que mon intervention initiale était pas très claire (même si ThomasN a compris), le code tel qu'il est me va très très bien pour bosser.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-togglable-area-bottom.patch 0001-togglable-area-bottom.patch ajouté
La zone de clic est limitée uniquement au chevron, je la verrais plus large.
Oui, c'est trop difficile de cliquer au bon endroit sans ça.
Aussi, pour rendre l'affaire un peu visible, le patch attaché double la bordure des pages "pliées".
~~
À l'usage, tout replié, je déplace une page et ça fait apparaitre une sous-page qui était cachée jusque là. (je n'ai pas regardé le code pour comprendre ce qu'il se passe là).
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0001-manager-toggle-pages-in-manager-home-27618.patch 0001-manager-toggle-pages-in-manager-home-27618.patch ajouté
avec la modif de Fred :)
À l'usage, tout replié, je déplace une page et ça fait apparaitre une sous-page qui était cachée jusque là. (je n'ai pas regardé le code pour comprendre ce qu'il se passe là).
Ca me paraît être un comportement normal, en tout cas c'est voulu de ma part: si tu ajoutes un enfant à une page repliée, alors soit tu masques l'enfant après le drop (mais ça fait bizarre), soit tu déplies le parent, pour avoir tous les enfants visibles ou cachés, et non un mix des deux.
Mis à jour par Frédéric Péters il y a environ 4 ans
Dans mon cas je déplaçais une page de premier niveau en la laissant de premier niveau et une page de second niveau sans rapport apparaissait (et une seule page, d'une branche qui en contient plusieurs).
- a
- b
- c (avec des enfants, A, B, C, D)
- d
- e
je drag/drop b pour la mettre derrière d,
- a
- c (toujours marqué plié)
- C
- d
- b
- e
J'en aurais fait une vidéo mais là je ne reproduis évidemment plus... j'imagine du coup que je devais avoir des données pas très carrées dans mes pages, genre d'anciens essais avec un ordre de page incohérent, ou de cycles entre pages parentes/enfantes, etc.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier page-collapse.png page-collapse.png ajouté
- Fichier 0001-collapse-child-pages.patch 0001-collapse-child-pages.patch ajouté
Pas très content de ma double bordure, je tente autre chose, si plutôt que cacher les pages enfant elles étaient juste réduites en hauteur, cf capture ?
(et patch pas abouti, ne gère pas le moment du drop dans une zone "pliée")
Mis à jour par Pierre Cros il y a environ 4 ans
Frédéric Péters a écrit :
Pas très content de ma double bordure, je tente autre chose, si plutôt que cacher les pages enfant elles étaient juste réduites en hauteur, cf capture ?
Mais comment on y accède à ces pages enfants sur ta capture ? La page "Test" semble dépliée et je ne vois aucun des enfants (enfin, je vois un amas indistinct d'enfants :), pas moyen de choisir une page). J'imagine que je rate un truc.
Mis à jour par Frédéric Péters il y a environ 4 ans
Ça n'a rien changé par rapport au patch précédent, les pages enfant sont dépliées en cliquant sur le ⌄.
Mis à jour par Pierre Cros il y a environ 4 ans
Ok, faudrait que j'évite donc de me mêler de ce genre de discussion sans avoir pu tester. Je pensais comprendre ce que le patch proposait, c'est sans doute pas le cas, oubliez-moi.
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier Capture d’écran de 2020-04-24 10-28-28.png Capture d’écran de 2020-04-24 10-28-28.png ajouté
Thomas J proposait d'indiquer le nombre d'enfants, ça pourrait donner qqch comme ça
(en améliorant le padding, le wording, etc)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier pages-doc.png pages-doc.png ajouté
En pratique et en tant que tel ce ticket (me) pose problème, simulé sur le site de la documentation ça donnerait la page attachée; de là, si je veux aller modifier une page je dois connaitre précisément la hiérarchie du site, ou passer du temps à fermer/ouvrir des sous-pages. (cela qu'on laisse poindre un peu des sous-pages ou pas).
Je me trouve ainsi à souhaiter un bouton "tout déplier", en étant presque sûr que je vais systématiquement cliquer dessus. Peut-être qu'avec le premier niveau déplié, ça passerait mieux, mais je crains écrire ça uniquement parce que ça me permet sur le site de la doc de voir directement la page "notes de mise à jour".
Mis à jour par Frédéric Péters il y a environ 4 ans
Je serais ici pour un retour à #27618#note-11 sans ma proposition de double bordure (i.e. le patch initial + une zone de clic plus large) + ne pas avoir ça actif par défaut, avoir un settings.MANAGE_HOME_COLLAPSE_PAGES pour l'activer; et je verrai pour l'activer sur doc-publik (et/ou ailleurs), ça permettra à Pierre d'expérimenter la modification.
Et qu'on prenne par ailleurs le temps et la méthode pour réfléchir au fonctionnement du backoffice combo (quand il y a un grand nombre de pages).
Mis à jour par Lauréline Guérin il y a environ 4 ans
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e181863b9f7e1ca7412c9bfe6b088fe59176126a
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Tue Apr 21 14:52:25 2020 +0200
manager: toggle pages in manager home (#27618)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: toggle pages in manager home (#27618)