Skip to content

Ajout de l'autofill pour le champ "lieunaissance"#117

Open
7ute wants to merge 1 commit into
LAB-MI:masterfrom
7ute:patch-autofill
Open

Ajout de l'autofill pour le champ "lieunaissance"#117
7ute wants to merge 1 commit into
LAB-MI:masterfrom
7ute:patch-autofill

Conversation

@7ute

@7ute 7ute commented Apr 30, 2020

Copy link
Copy Markdown

Ajout d'un autofill-scope au champ de saisie du lieu de naissance, pour éviter la re-saisie à chaque changement de page.

Ajout d'un [autofill-scope](https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#autofill-scope) au champ de saisie du lieu de naissance, pour éviter la re-saisie à chaque changement de page.

@OlivierGuinart OlivierGuinart left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

On vote où pour que ce changement monte dans master ?

Comment thread src/index.html
@cClaude

cClaude commented May 3, 2020

Copy link
Copy Markdown

Cette pull-request est un duplicate de la pull-request #85
La solution #85 consistant reprendre tous les champs me semble plus pertinente.

@7ute

7ute commented May 3, 2020

Copy link
Copy Markdown
Author

@cClaude Je ne pense pas que ce soit un duplicate, vu que l'utilisation de l'auto-fill comme du localStorage ne me semblent pas exclusifs.

Cette PR adresse le cas d'utilisation actuel :

  • on rempli l'attestation
  • on génère une attestation
  • on revient à la page précédente
  • tous les champs sont pré-remplis par l'auto-fill (y compris lieunaissance)

L'ajout d'un enregistrement localStorage adresse le cas d'usage suivant :

  • on rempli l'attestation
  • on clic sur le bouton d'enregistrement des données
  • on génère l'attestation
  • on retourne à la page précédence par un autre biais que le bouton retour (rafraichissement de page, accès direct)
  • on peut préremplir le formulaire avec les données enregistrées précédemment.

@OlivierGuinart

OlivierGuinart commented May 3, 2020 via email

Copy link
Copy Markdown

@Kerenok

Kerenok commented Nov 4, 2020

Copy link
Copy Markdown

Pourquoi ce simple oubli n'a-t-il pas été corrigé avant la mise en production de la version Automne-Hiver? C'est à se demander si les développeurs utilisent vraiment leur application!

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.

4 participants