UX/UI: dégriser la modale de transfert de candidature

[EwenKorr]

🤔 Pourquoi ?

Lorsque je transfère une candidature, la modale de confirmation est grisée et non cliquable, je dois faire un retour navigateur pour avoir de nouveau accès au site.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

Déplacer la génération du HTML des modales plus haut dans le DOM, et donc plus haut dans l'arborescence des templates.
@hellodeloo m'a pointé vers le projet le-marché où un {% block modals %} existe, tout comme nous avons un {% block scripts %}.

Si je ne me trompe pas :

1. on ne peut pas utiliser de {% block * %} dans les fichiers html inclus avec {% include * %}
2. l'arborescence des templates est la suivante :

 base.html
└ process_base.html
    └ process_details_company.html  # {% block modals %} à mettre ici
        └ <une série d'includes ici> 

🚨 À vérifier

Ça ne gêne pas d'autres pages ? Je ne suis pas encore familier avec l'organisation des templates.

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

Un quickfix a été déployé par David dans le thème : enlever le sticky du bandeau titre. Pour tester, j'ai donc utilisé le thème version 2.1.0.

itou/utils/staticfiles.py

…
    "theme-inclusion": {
        "download": {
            "url": "https://github.com/gip-inclusion/itou-theme/archive/refs/tags/v2.1.0.zip",
            "sha256": "bee559639f0a2c3f91d19d474cd10c4413a3775136830c6f240deddfa61a655c",
        },
        "extract": {
            "origin": "itou-theme-2.1.0/dist",
            "destination": "vendor/theme-inclusion/",
            "files": [
…

💻 Captures d'écran

Avant :

Après :

[EwenKorr]

N'hésitez pas à me mettre sur la bonne voie en termes de bonnes pratiques ! :)

[notion-workspace]

🔁 [Retour de recette menu à gauche] La modale de transfert de candidature est grisée

[EwenKorr]

Merci pour les retours Xavier !

Deux questions sur git/github :

• j'ai fait un git rebase --interactive master, comme précisé dans la doc. Ce n'est peut-être pas hyper utile dans le cas de petites PR comme ça ? Ça pollue même peut-être plus qu'autre chose ?
• j'ai corrigé dans un commit séparé l'id non unique. Je ferai un git rebase --interactive master pour n'avoir que 2 jolis commits avant le merge de Github. Ce merge va bien garder les 2 commits séparés dans l'historique ?

[xavfernandez]

Alors j'ai l'impression que tu as récupéré d'autres commits d'une autre PR sur celle-ci.

Le git rebase --interactive master est assez pratique pour retravailler les commits donc je 👍

Tu as aussi la possibilité de faire des commits de fixup: https://git-scm.com/docs/git-commit/2.32.0#Documentation/git-commit.txt---fixupamendrewordltcommitgt qui pourront être facilement intégrer au commit correspondant

Pour l'historique du repo/des commits on essaie d'avoir des petits commits qui ne font qu'une seule chose (dans la mesure du possible).
Les PRs sont mergées via une merge queue et la stratégie "rebase and merge" donc l'ensemble des commits de la PR sont conservés tels qu'ils sont au moment où la PR rejoint la merge queue.

[hellodeloo]

LGTM

[calummackervoy]

Il me paraît que la modale nouveautés peut avoir un nouvelle maison mais content du merger de toute façon

[calummackervoy]

Je vois quecette classe est désormais incluse dans itou-theme 👍

[hellodeloo]

Oui monsieur ;)

[calummackervoy]

On pourrait bouger cette modale au block modals maintenant je pense ?

[hellodeloo]

Tu penses bien mais pas la peine de faire ça maintenant.
Je vais céer un ticket pour bouger toutes les modales dans block modals. ... plus tard