Projet

Général

Profil

Development #58105

attributs sur les champs du formulaire de modification de mot de passe

Ajouté par Frédéric Péters il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
22 octobre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

De https://web.dev/sign-in-form-best-practices/#new-password :

For password inputs, use the appropriate autocomplete and id values to help browsers differentiate between new and current passwords.

Use autocomplete="new-password" and id="new-password" for the password input in a sign-up form, or the new password in a change-password form.
Use autocomplete="current-password" and id="current-password" for the password input in a sign-in form, or the input for the user's old password in a change-password form. This tells the browser that you want it to use the current password that it has stored for the site.

(je serais pour ignorer la partie sur l'attribut id, qui n'est de toute façon pas applicable sur le champ de confirmation vu que ça donnerait un id dupliqué).


Fichiers

Révisions associées

Révision 25f0a775 (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

misc: add attribute to current password field on change password page (#58105)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 2 ans

+    old_password.widget.attrs.update({'autocomplete': 'current-password'})

Perso je n'avais pas encore vu cette manière de faire, elle vient de https://docs.djangoproject.com/en/2.2/ref/forms/widgets/#styling-widget-instances (elle n'était pas dans la doc de 1.11), j'ai trouvé ça plus sympa qu'avoir à définir le widget totalement, façon

from .widgets import PasswordInput
[...]
old_password = PasswordField(label=_('Old password'), widget=PasswordInput(attrs={'autocomplete': 'current-password'}))
#2

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée

Oui ça m'a l'air plus propre.

#3

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 25f0a77573ac056d652f0c3088532672fb4f7142
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Oct 23 09:57:48 2021 +0200

    misc: add attribute to current password field on change password page (#58105)
#4

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF