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

Fix issue #73 date de naissance #130

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

gafl
Copy link

@gafl gafl commented Nov 4, 2020

Comme decrit dans issue #73, les chiffres entrés pour date du mois, ou mois d'année sont reportés dans le champ suivant (mois ou annee) sur certains navigateurs mobiles. Par exemple, en entrant 15 puis 0, le champ affiche 15/150, au lieu de 15/0. Problème reproduit et fixe verifiée sur Samsung Internet v12.1.4.3 et Chrome v84.0.4147.89 sur Samsung Galaxy S4 et S5. Non regression verifiée sur Chrome 86.0.4240.111 et FireFox 82.0.2 sur PC Windows 10.

Les chiffres entres pour date du mois, ou mois d'annee sont reportes
dans le champ suivant (mois ou annee). Par exemple, en entrant
01 puis 0, le champ affiche 01/010.
@gafl gafl mentioned this pull request Nov 4, 2020
@maximebochon
Copy link

maximebochon commented Nov 22, 2020

Je viens de tester cette proposition mais je constate une régression, que ce soit sur Chromium sous Linux, sur Chrome sous Windows 10, sur Firefox sous Linux et Windows 10, ou sur Safari sous iOS 14.2. Voici comment reproduire cette régression :

  • se placer dans le champ "Date de naissance"
  • si ce champ n'est pas vide, sélectionner tout son contenu et le supprimer
  • saisir deux chiffres : une barre oblique de séparation est ajoutée automatiquement
  • supprimer la barre oblique à l'aide de la touche "retour arrière" : la suppression n'a pas lieu
  • saisir deux chiffres : une seconde barre oblique de séparation est ajoutée automatiquement
  • supprimer la seconde barre oblique à l'aide de la touche "retour arrière" : la suppression n'a pas lieu

Voici l'environnement de démo que j'ai utilisé : http://test.bochon.net/covid/naissance-input-event-v1/

Il est probable que l'événement input de cette proposition fausse la position du curseur par rapport à l'actuel keyup et compromet l'algorithme d'assistance à la saisie de la date de naissance. Ce correctif n'est donc pas recevable en l'état. D'ailleurs je pense qu'il faudrait aller plus loin dans l'analyse car pour l'instant aucun diagnostique réel du problème n'a été posé.

Pour autant on peut contourner le problème de deux manières relativement simples :

  1. désactiver l'assistance à la saisie (car c'est elle qui semble poser problème ici)
  2. utiliser un champ HTML 5 de type date, comme c'est fait pour la date de sortie

Étant partisan de la solution 2, en voici une démo : http://test.bochon.net/covid/naissance-html5-date-v1/
Patch associé : http://test.bochon.net/covid/naissance-html5-date-v1/patch.diff

@Papy-G
Copy link

Papy-G commented Nov 26, 2020

Le champ de date de sortie n'est pas l'exemple à suivre, car il apparaît invalide sous de nombreux navigateurs (Mozzilla/OSX), et empêche parfois la génération de l'attestation.

Par ailleurs, son format n'est pas le même que celui de la date, ce qu'il serait souhaitable d'unifier.

@Papy-G
Copy link

Papy-G commented Nov 26, 2020

Confirmé avec la version de test, à présent, les deux champs restent entourés de rouge après saisie.

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.

3 participants