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

[GEN-191] Affichage des boutons enregistrer en haut des formulaires d'admin #3965

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

leo-naeka
Copy link
Contributor

@leo-naeka leo-naeka commented Apr 22, 2024

🤔 Pourquoi ?

Le formulaire d'édition peut être très long en admin et la plupart des actions se font en partie haute.
Évite de scroll tout en bas du form.

https://www.notion.so/plateforme-inclusion/Ajout-d-un-bouton-enregistrer-dans-l-admin-1385ab2419b0439c92ef1d42cc7a1fe3?pvs=4

🍰 Comment ?

Django prévoit le coup avec ModelAdmin.save_on_top mais ne fournit pas de settings pour le configurer par défaut.

Monkey patch de ModelAdmin au démarrage de l'app (la surchage du template ne s'y prête pas bien).

→ Ajout de l'option à ItouModelAdmin

💻 Captures d'écran

Avant

Capture d’écran 2024-04-22 à 15 10 44

Après

Capture d’écran 2024-04-22 à 15 11 06

🚨 À vérifier

  • Ajouter l'étiquette « no-changelog » ?
  • Mettre à jour le CHANGELOG_breaking_changes.md ?

@leo-naeka leo-naeka self-assigned this Apr 22, 2024
Copy link

@xavfernandez
Copy link
Contributor

👍 Je ne connaissais pas cette option.

Par contre, plutôt que le monkeypatch, peut-être tout simplement le setter sur notre ItouModelAdmin qui devrait a priori être utilisé sur tous nos modèles ?

@leo-naeka
Copy link
Contributor Author

👍 Je ne connaissais pas cette option.

Par contre, plutôt que le monkeypatch, peut-être tout simplement le setter sur notre ItouModelAdmin qui devrait a priori être utilisé sur tous nos modèles ?

C'est pas le cas pour les QPV qui utilisent GISModelAdmin (mais 3 champs donc pas utile, je laisse en état).
Ok pour set sur ItouModelAdmin 👍
Je voulais surtout être au plus près de Django si toutefois d'autres éléments de l'admin ne se basaient pas sur ItouModelAdmin à l'avenir (car ce n'est peut-être pas le genre d'option auquel l'on pensera lors de reviews).

@leo-naeka leo-naeka force-pushed the leo/show-admin-save-buttons-on-top branch from f5a35ac to 9bb9468 Compare April 22, 2024 14:25
Copy link
Contributor

@rsebille rsebille left a comment

Choose a reason for hiding this comment

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

Simple, efficace 👌.
Pour QPVAdmin() vu qu'il y a un mixin GeoModelAdminMixin ça pourrais valoir le coup de changer l'héritage en GeoModelAdminMixin, ItouModelAdmin, ça permettrais d'ajouter ensuite un petit check pour obliger l'utilisation de ItouModelAdmin pour tout nos modèles 😁.

@leo-naeka leo-naeka added this pull request to the merge queue Apr 25, 2024
Merged via the queue into master with commit cd2c507 Apr 25, 2024
10 checks passed
@leo-naeka leo-naeka deleted the leo/show-admin-save-buttons-on-top branch April 25, 2024 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants