Project

General

Profile

Development #14939

avoir un template pour la pagination des objets dans un listing

Added by Serghei Mihai about 7 years ago. Updated about 5 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
09 February 2017
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Comme plusieurs backoffices (authentic, corbo, ...) font la pagination de certains de ses objets on peut deplacer la logique dans gadjo.


Files

Associated revisions

Revision ef4a43df (diff)
Added by Frédéric Péters about 6 years ago

add generic pagination template (#14939)

History

#1

Updated by Frédéric Péters about 6 years ago

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.

#2

Updated by Frédéric Péters about 6 years ago

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).

#3

Updated by Thomas Noël about 6 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> ?)

#4

Updated by Frédéric Péters about 6 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.

#5

Updated by Frédéric Péters about 5 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF