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

Migration invitation usager et reset password agent au DSFR #4926

Open
wants to merge 5 commits into
base: production
Choose a base branch
from

Conversation

AntoineGirard
Copy link
Member

@AntoineGirard AntoineGirard commented Dec 24, 2024

Contexte

Dans le cadre de la migration au DSFR, on migre les formulaires suivants :

  • Réponse à l’invitation des usagers par les agents
  • Demande de réinitialisation de mot de passe agent
  • Réinitialisation de mot de passe agent

Solution

En faisant ça, on peut se séparer dans l’ancienne logique permettant d’afficher/cacher le mot de passe dans le champ.
On a également le nouveau de système d’aide dynamique au choix de mot de passe.

Captures d'écran

Avant Après
image image
image image
image image

@AntoineGirard AntoineGirard marked this pull request as ready for review December 24, 2024 14:54
Copy link

@Teodora-Stanki Teodora-Stanki left a comment

Choose a reason for hiding this comment

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

  • On a un espace qui s'est introduit pour le champ "mot de passe" au dsfr, entre le libellé et le champ. Est-il possible de le réduire pour avoir le meme espacement que les autres champs ?
  • On a perdu le hint du champ du telephone, il faudrait le reprendre et l'insérer en "texte de description additionnel" entre le libellé et le champ.
    Le reste est ok

@AntoineGirard
Copy link
Member Author

On a un espace qui s'est introduit pour le champ "mot de passe" au dsfr, entre le libellé et le champ. Est-il possible de le réduire pour avoir le meme espacement que les autres champs ?

@Teodora-Stanki effectivement
Cet espace est généré par l’ajout de la case qui affiche/cache l’affichage du mot de passe
Vu que j’utilise un template commun, on a le même problème partout
Je vais essayer de corriger, sinon je ferai ça dans une PR séparée

On a perdu le hint du champ du telephone, il faudrait le reprendre et l'insérer en "texte de description additionnel" entre le libellé et le champ.

Effectivement, my bad
Je vais le remettre. D’ailleurs, on pourra peut-être regarder pour ajuster ce wording parce qu’il ne me semble pas qu’on puisse envoyer des SMS sur les numéros internationaux

@AntoineGirard
Copy link
Member Author

Et voilà @Teodora-Stanki
image

Copy link
Contributor

@adipasquale adipasquale left a comment

Choose a reason for hiding this comment

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

magnifique cette PR ça fait plaisir et ça va améliorer l’UX pour tout le monde 😻

j’ai testé en local les 3 pages, essayé de faire apparaître les erreurs, testé sans JS aussi ✅

= f.password_field :password, as: :password, label: "Mot de passe", class: "form-control", autocomplete: "new-password", required: true, id: "password"
span.fa.fa-fw.fa-eye.toggle-password role="button" tabindex="0"

= render "agents/mot_de_passes/new_password_hints"
Copy link
Contributor

Choose a reason for hiding this comment

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

plus qu’une utilisation de ce partial dans agents/mot_de_passes/edit.html.slim, on pourrait éventuellement l’y inliner mais ce n’est absolument pas bloquant, je le note juste au passage

= render "agents/mot_de_passes/new_password_hints"
.rdv-text-align-center= f.button :submit, t("devise.invitations.edit.submit_button")
.fr-col-md-12= f.dsfr_email_field :email, required: true
.fr-col-md-12= f.dsfr_phone_field :phone_number, hint: "Saisissez un numéro à 10 chiffres de France métropole ou d’outre-mer, ou bien un numéro international avec le préfixe du pays."
Copy link
Contributor

Choose a reason for hiding this comment

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

je dis vive l’inlining 👍 est-ce que tu penses que la clé i18n simple_form.hints.user.phone_number est encore utilisée ? sinon je vote pour la supprimer au passage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🚀 To deploy
Development

Successfully merging this pull request may close these issues.

3 participants