Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup : homogénéiser la pagination #825

Merged
merged 7 commits into from
Apr 21, 2023
Merged

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Apr 18, 2023

Quoi ?

Homogénéisé l'usage de la pagination dans certains controlleurs :

  • controlleurs concernés : AdminController, ShiftFreeLog, MembershipShiftExemption, HelloassoController, RegistrationController, AmbassadorController
  • généraliser l'usage de page_count ; result_count ; current_page

Informations supplémentaires

  • i'ai essayé au début de sortir le <ul class="pagination" dans un template dédié et l'importer là où l'on en a besoin, mais ça semble être plus compliqué qu'imaginais... il y a du js, des route qui changent, et des fonctionnement légèrement différent
  • la pagination reste "simpliste" dans la plupart des cas, si il y a 50 pages ca les affiche tous (sauf sur "Liste des membres" qui a des règles évoluées / complexes)

@symartin
Copy link
Collaborator

Hello,

c'est plus clean, et le fait de mettre la page dans le formulaire du filtre va faire plaisir a bcp de monde 😃 (et cela enlève le bug de qd on est sur la dernière page et que l'on change le filtre).

Mais dans certain cas, cela à un comportement bizarre. Par exemple dans /helloasso/payments comme il y a 30 pages, on a le chevron qui revient à la ligne:
paiements Helloasso pagination

ce qui n'est pas homogène avec la façon don cela est géré sur /admin/users, ou il y a ... pour réduire le nombre de page (alors qu'il y en a que 11):
admin_users.

Enfin, pas tres grave, mais si on met à la main une page plus grande que le maximum, cela nous donne une page vide :
paiements Helloasso pagination 2.

@raphodn
Copy link
Member Author

raphodn commented Apr 20, 2023

ce qui n'est pas homogène avec la façon don cela est géré sur /admin/users, ou il y a ... pour réduire le nombre de page (alors qu'il y en a que 11):

oui carrément, mais je trouve la gestion des "..." plutôt complexe dans /admin/users. donc je n'ai pas (encore) généralisé. il faudrait qu'on trouve une solution long terme. qui sera de sortir la pagination dans un template dédié, mais je n'arrive pas à voir comment le faire simplement :(

Copy link
Collaborator

@symartin symartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le seul que je n'ai pas pu tester c'est le "campagne HelloAsso"

@raphodn raphodn merged commit fff10ac into master Apr 21, 2023
@raphodn raphodn deleted the raphodn/cleanup-pagination branch April 21, 2023 09:34
OursDesCavernes pushed a commit to Les400Coop/gestion-compte that referenced this pull request Jan 20, 2024
* Cleanup AdminController (user list)

* Cleanup & fix ShiftFreeLog

* Cleanup & fix MembershipShiftExemption

* Cleanup AmbassadorController

* Cleanup HelloAssoController

* Cleanup RegistrationController
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants