Development #14939
avoir un template pour la pagination des objets dans un listing
0%
Description
Comme plusieurs backoffices (authentic, corbo, ...) font la pagination de certains de ses objets on peut deplacer la logique dans gadjo.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-add-generic-pagination-template-14939.patch 0001-add-generic-pagination-template-14939.patch ajouté
- Fichier 0001-manager-use-gadjo-template-for-pagination-14939.patch 0001-manager-use-gadjo-template-for-pagination-14939.patch ajouté
- Fichier 0001-lingo-use-gadjo-template-for-pagination-14939.patch 0001-lingo-use-gadjo-template-for-pagination-14939.patch ajouté
- Fichier 0001-general-use-gadjo-template-for-pagination-14939.patch 0001-general-use-gadjo-template-for-pagination-14939.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Testé en local en remplacement dans combo, corbo et authentic.
Ça reprend le style qui était partagé par Authentic et Corbo; ça reprend aussi le templatetag "querystring" de django-tables2.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-misc-use-gadjo-template-for-pagination-14939.patch 0001-misc-use-gadjo-template-for-pagination-14939.patch ajouté
- Fichier 0001-add-generic-pagination-template-14939.patch 0001-add-generic-pagination-template-14939.patch ajouté
Patch pour passerelle (pour les logs et la liste des "API users").
Et nouveau patch gadjo qui contient la possibilité de préciser l'ancre à utiliser (pour les logs, pour pointer la page sur #logs). Et aussi un changement au style pour que la page active soit avec un fond gris, plus discret que l'actuel bleu. (il y aura d'autres évolutions au template, mais à discuter dans de futurs tickets).
Mis à jour par Thomas Noël il y a plus de 6 ans
Dans le template, ligne 33, Il y a un {% if page_obj.paginator.num_pages > 1 %}
alors qu'on est déjà à l'intérieur d'un {% if page_obj.paginator.num_pages > 1 %}
.
À part ce détail, le patch sur gadjo me semble ok.
Sur l'usage dans les autres, ok (on est d'accord que les templates dans passerelle était merdiques, avec des </div> pour fermer des <p> ?)
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
Dans le template, ligne 33, Il y a un {% if page_obj.paginator.num_pages > 1 } alors qu'on est déjà à l'intérieur d'un { if page_obj.paginator.num_pages > 1 %}.
Cool, merci.
commit ef4a43df28d24b949fe64862c1eeb1b152717982 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Dec 26 20:52:48 2017 +0100 add generic pagination template (#14939)
Sur l'usage dans les autres, ok (on est d'accord que les templates dans passerelle était merdiques, avec des </div> pour fermer des <p> ?)
On est d'accord.
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
add generic pagination template (#14939)