Project

General

Profile

Actions
Copy link

Développement #14939

closed

avoir un template pour la pagination des objets dans un listing

Added by Serghei Mihai almost 9 years ago. Updated about 7 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

Download all files
0001-add-generic-pagination-template-14939.patch (5.57 KB) 0001-add-generic-pagination-template-14939.patch Frédéric Péters, 26 December 2017 08:53 PM
0001-manager-use-gadjo-template-for-pagination-14939.patch (2.07 KB) 0001-manager-use-gadjo-template-for-pagination-14939.patch authentic Frédéric Péters, 26 December 2017 08:56 PM
0001-lingo-use-gadjo-template-for-pagination-14939.patch (1.35 KB) 0001-lingo-use-gadjo-template-for-pagination-14939.patch combo Frédéric Péters, 26 December 2017 08:56 PM
0001-general-use-gadjo-template-for-pagination-14939.patch (2.91 KB) 0001-general-use-gadjo-template-for-pagination-14939.patch corbo Frédéric Péters, 26 December 2017 08:57 PM
0001-misc-use-gadjo-template-for-pagination-14939.patch (2.46 KB) 0001-misc-use-gadjo-template-for-pagination-14939.patch passerelle Frédéric Péters, 26 December 2017 10:33 PM
0001-add-generic-pagination-template-14939.patch (6.2 KB) 0001-add-generic-pagination-template-14939.patch Frédéric Péters, 26 December 2017 10:36 PM
Actions
Copy link Download all files
 #1

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

Actions
Copy link Download all files
 #2

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

Actions
Copy link
 #3

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

Actions
Copy link
 #4

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

Actions
Copy link
 #5

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

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

Also available in: Atom PDF