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

ETQ usager, j'ai une indication sur le format de date à saisir en fonction de mon navigateur #10709

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

colinux
Copy link
Member

@colinux colinux commented Aug 22, 2024

Même principe que le "vrai" champ date, du JS vient remplacer le format du hint si nécessaire
Capture d’écran 2024-08-22 à 11 07 26

Copy link

codecov bot commented Aug 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.63%. Comparing base (bd4c316) to head (e46f09d).
Report is 91 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10709      +/-   ##
==========================================
- Coverage   84.63%   84.63%   -0.01%     
==========================================
  Files        1117     1118       +1     
  Lines       24730    24738       +8     
  Branches     4598     4600       +2     
==========================================
+ Hits        20930    20936       +6     
- Misses       3800     3802       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@colinux colinux added this pull request to the merge queue Aug 22, 2024
@tchak tchak removed this pull request from the merge queue due to a manual request Aug 22, 2024
@tchak
Copy link
Member

tchak commented Aug 22, 2024

Ça ne marche pas car le format est dépendant de la locale du navigateur/os qui n'est pas forcément la même que celle de la locale du site. Nous avons eu des problèmes avec ça dans le passé. Il faut utiliser date_input_hint_controller

@tchak
Copy link
Member

tchak commented Aug 22, 2024

C'est d'ailleurs probablement la raison pour laquelle les utilisateurs se plantent. Une fois sortie la tête des champs, je pense intégrer le date picker de react-aria-components qui sera bien localisé en fonction de la locale du site et sera plus accessible.

@tchak tchak added this pull request to the merge queue Aug 22, 2024
@tchak
Copy link
Member

tchak commented Aug 22, 2024

ah mais t'a bien fait, je ne sais juste pas lire

Merged via the queue into demarches-simplifiees:main with commit 6a642c3 Aug 22, 2024
18 checks passed
@colinux colinux deleted the cojo-date-hint-localized branch August 22, 2024 10:41
@colinux colinux added facile Facile à résoudre (mais pas forcément prioritaire) a-communiquer À communiquer aux tickets HelpScout mentionnés ou sur FeatureUpvote engagement Issue sur laquelle on s'est engagé en externe accessibilité feature:avis contributions acceptées Issue ouverte aux contributions externes archive-2017 Issue antérieures à 2017 fermées automatiquement archive-2018 Issue antérieures à 2018 fermées automatiquement AIPD administrateurs a-communiquer-instances labels Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-communiquer À communiquer aux tickets HelpScout mentionnés ou sur FeatureUpvote a-communiquer-instances accessibilité administrateurs AIPD archive-2017 Issue antérieures à 2017 fermées automatiquement archive-2018 Issue antérieures à 2018 fermées automatiquement contributions acceptées Issue ouverte aux contributions externes engagement Issue sur laquelle on s'est engagé en externe facile Facile à résoudre (mais pas forcément prioritaire) feature:avis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants