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.
Files
Associated revisions
History
Updated by Frédéric Péters almost 7 years ago
- File 0001-add-generic-pagination-template-14939.patch 0001-add-generic-pagination-template-14939.patch added
- File 0001-manager-use-gadjo-template-for-pagination-14939.patch 0001-manager-use-gadjo-template-for-pagination-14939.patch added
- File 0001-lingo-use-gadjo-template-for-pagination-14939.patch 0001-lingo-use-gadjo-template-for-pagination-14939.patch added
- File 0001-general-use-gadjo-template-for-pagination-14939.patch 0001-general-use-gadjo-template-for-pagination-14939.patch added
- Status changed from Nouveau to En cours
- Patch proposed changed from No to Yes
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.
Updated by Frédéric Péters almost 7 years ago
- File 0001-misc-use-gadjo-template-for-pagination-14939.patch 0001-misc-use-gadjo-template-for-pagination-14939.patch added
- File 0001-add-generic-pagination-template-14939.patch 0001-add-generic-pagination-template-14939.patch added
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).
Updated by Thomas Noël almost 7 years ago
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> ?)
Updated by Frédéric Péters almost 7 years ago
- Status changed from En cours to 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.
Updated by Frédéric Péters almost 6 years ago
- Status changed from Résolu (à déployer) to Solution déployée
add generic pagination template (#14939)