UX : Système de pagination sur page dataset perturbant

etalab / transport-site

Rendre disponible, valoriser et améliorer les données transports

https://transport.data.gouv.fr

184 stars 28 forks source link

UX : Système de pagination sur page dataset perturbant #4023

Closed cyrilmorin closed 1 day ago

cyrilmorin commented 1 week ago

La pagination sur la page dataset pose problème car on pense parfois avoir tout vu (à tort) à cause des n° de pages situés tout en bas de l'écran alors que le nb de cards est limité à 10

Propositions : 1- Supprimer les système de pagination (en ne chargeant les briques que lorsque l'on scroll vers le bas ?) 2- Augmenter le nombre de briques à 20 (voire proposer les options de nombres de briques à charger (ex : 10, 20, 50)) 3- Afficher le nombre de page en haut et en bas de la liste

Faites au mieux :)

Merci,

Cyril

vdegove commented 5 days ago

Est-ce que simplement remonter la liste des pages t’irait ? Comme ça :

cyrilmorin commented 4 days ago

Hello @vdegove , Oui j'y a avais pensé en mode quick win, ce serait déjà mieux :) On pourrait à minima augmenter le nb de dataset à 20 si ce n'impacte pas les perf ?

Sinon OK pour moi !

vdegove commented 4 days ago

Alors, j’ai fait le 1er (changer l’emplacement du bloc de pagination), je suis sur le 2ème (changer le nombre d’items renvoyés).

Le problème c’est que le PAN a une config unique pour la pagination, qui vaut pour tous les endroits où on pagine des trucs. Changer à 20 items le changerait sur tous les endroits suivants :

Liste de datasets (recherche)
Liste de contacts dans le backoffice
Liste de datasets dans le backoffice
Liste d’erreurs sur la page de détail d’une resource
Liste d’erreurs sur la page spécifique de validation (je sais même plus si ce template est vraiment utilisé, ahah)

T’es ok avec ça ?

Exemple, ici

Du peu que j’ai pu tester, ça n’affecte pas la performance, mais c’est du test manuel, je ne suis pas outillé pour ça.

AntoineAugusti commented 4 days ago

Le problème c’est que le PAN a une config unique pour la pagination, qui vaut pour tous les endroits où on pagine des trucs.

On peut adapter notre code existante pour que Scrivener, l'outil de pagination, ait une configuration différente selon l'usage je pense. À voir si c'est OK de passer à 20 partout.

cyrilmorin commented 1 day ago

@vdegove je trouve que cela apporte un plus également pour les autres pages concernées. Dans le même esprit, si on a une liste à rallonge, on a toujours intérêt à voir un peu plus d'item pour chaque liste

=> Banco !