spip
/
spip
32
14
Fork 15
Code Issues 435 Pull Requests 13 Projects Releases Wiki Activity
Labels Milestones
New Issue

Datepicker : aujourd'hui ne ramène pas à aujourd'hui #5194

Closed
opened 10 months ago by maieul · 12 comments
maieul commented 10 months ago
Collaborator

Soit un article datée du 30 juin 2022.

Si je veux modifier la date, et que je click sur "aujourd'hui" dans le date picker, cela me selectionne le ... 30 juin 2022, et pas la date d'aujourd'hui.

Soit un article datée du 30 juin 2022. Si je veux modifier la date, et que je click sur "aujourd'hui" dans le date picker, cela me selectionne le ... 30 juin 2022, et pas la date d'aujourd'hui.
b_b added the
formulaires
bug
 labels 10 months ago
b_b commented 10 months ago
Owner

Version cible ?

Version cible ?
maieul commented 10 months ago
Poster
Collaborator

Des que possible dirai je. En tt cas le bug est la en 4.0, et je crois aussi en master

Des que possible dirai je. En tt cas le bug est la en 4.0, et je crois aussi en master
b_b added this to the 4.0 milestone 10 months ago
b_b commented 10 months ago
Owner

Je vois d'après la doc https://bootstrap-datepicker.readthedocs.io/en/stable/options.html?highlight=today#todaybtn que ce bouton peut avoir deux comportements, linked et unlinked cf :

If true or “linked”, displays a “Today” button at the bottom of the datepicker to select the current date. If true, the “Today” button will only move the current date into view; if “linked”, the current date will also be selected.

On utilise l'option avec uniquement true cf 21097aece8/prive/formulaires/dateur/inc-dateur.html (L27) donc unlinked, la solution serait de passer l'option à linked.

@cerdic un avis ?

Je vois d'après la doc https://bootstrap-datepicker.readthedocs.io/en/stable/options.html?highlight=today#todaybtn que ce bouton peut avoir deux comportements, linked et unlinked cf : > If true or “linked”, displays a “Today” button at the bottom of the datepicker to select the current date. If true, the “Today” button will only move the current date into view; if “linked”, the current date will also be selected. On utilise l'option avec uniquement true cf https://git.spip.net/SPIP/spip/src/commit/21097aece89c1bcb9fed25a3ce0a1effdc792c49/prive/formulaires/dateur/inc-dateur.html#L27 donc unlinked, la solution serait de passer l'option à linked. @cerdic un avis ?
cerdic commented 10 months ago
Owner

Je crois que déjà le ticket est mal formulé, car si je clique sur le bouton "Aujourd'hui" ça ramène bien l'affichage du picker (on repasse sur le mois en cours, qui contient la date du jour).

Mais, de fait, la date du jour n'est pas sélectionnée.

Je suis retourné voir en 3.2 comment ça fonctionnait, et il n'y avait pas de bouton "aujourd'hui" ce qui est plus simple.

J'ai testé en changeant le comportement du bouton, mais ça m'emballe pas trop non plus car ça selectionne la date du jour ET ferme le calendrier.

Ça peut-être pertinent dans certains cas, mais pas tout le temps.

Par ailleurs le comportement par défaut actuel du picker est modifiable en ajoutant un attribut sur le input:

data-todayBtn="linked"

Donc j'aurais tendance à penser qu'on laisse le comportement par défaut actuel dans le picker.

Cependant, ce qui améliorerait certainement la compréhension de l'interface c'est que la date du jour soit highlightée d'une façon ou d'une autre, ce que faisait l'ancien picker.

Je crois que déjà le ticket est mal formulé, car si je clique sur le bouton "Aujourd'hui" ça ramène bien l'affichage du picker (on repasse sur le mois en cours, qui contient la date du jour). Mais, de fait, la date du jour n'est pas sélectionnée. Je suis retourné voir en 3.2 comment ça fonctionnait, et il n'y avait pas de bouton "aujourd'hui" ce qui est plus simple. J'ai testé en changeant le comportement du bouton, mais ça m'emballe pas trop non plus car ça selectionne la date du jour ET ferme le calendrier. Ça peut-être pertinent dans certains cas, mais pas tout le temps. Par ailleurs le comportement par défaut actuel du picker est modifiable en ajoutant un attribut sur le input: ``` data-todayBtn="linked" ``` Donc j'aurais tendance à penser qu'on laisse le comportement par défaut actuel dans le picker. Cependant, ce qui améliorerait certainement la compréhension de l'interface c'est que la date du jour soit highlightée d'une façon ou d'une autre, ce que faisait l'ancien picker.
b_b commented 10 months ago
Owner

Je suis retourné voir en 3.2 comment ça fonctionnait, et il n'y avait pas de bouton "aujourd'hui" ce qui est plus simple.

J'ai fait de même :)

Donc j'aurais tendance à penser qu'on laisse le comportement par défaut actuel dans le picker.

Ha ben mince, je viens jsute d'envoyer une PR pour ça, je vous laisse la fermer.

> Je suis retourné voir en 3.2 comment ça fonctionnait, et il n'y avait pas de bouton "aujourd'hui" ce qui est plus simple. > J'ai fait de même :) > Donc j'aurais tendance à penser qu'on laisse le comportement par défaut actuel dans le picker. > Ha ben mince, je viens jsute d'envoyer une PR pour ça, je vous laisse la fermer.
maieul commented 10 months ago
Poster
Collaborator

Ah oui, effectivement. Il n'empeche que

  1. Le "aujourd'hui" est ambigue, car tout le reste concerne des dates selectionnées, du coup pour moi c'était une selection.
  2. Peut être pour limiter l'ambiguité changer le libellé en "Voir / Aller à aujourd'hui"
  3. Et effectivement peut être une legère valorisation de la date du jour (mais il ne faudrait pas confondre avec la date selectionnée) serait positif
Ah oui, effectivement. Il n'empeche que 1. Le "aujourd'hui" est ambigue, car tout le reste concerne des dates selectionnées, du coup pour moi c'était une selection. 2. Peut être pour limiter l'ambiguité changer le libellé en "Voir / Aller à aujourd'hui" 3. Et effectivement peut être une legère valorisation de la date du jour (mais il ne faudrait pas confondre avec la date selectionnée) serait positif
b_b commented 10 months ago
Owner

Cependant, ce qui améliorerait certainement la compréhension de l'interface c'est que la date du jour soit highlightée d'une façon ou d'une autre, ce que faisait l'ancien picker.

Pour ça, il y a l'option todayHighlight cf https://bootstrap-datepicker.readthedocs.io/en/stable/options.html?highlight=today#todayhighlight

> Cependant, ce qui améliorerait certainement la compréhension de l'interface c'est que la date du jour soit highlightée d'une façon ou d'une autre, ce que faisait l'ancien picker. Pour ça, il y a l'option `todayHighlight` cf https://bootstrap-datepicker.readthedocs.io/en/stable/options.html?highlight=today#todayhighlight
cerdic commented 10 months ago
Owner

Je comprends pas ta phrase @maieul "Le "aujourd'hui" est ambigue, car tout le reste concerne des dates selectionnées, du coup pour moi c'était une selection."

Au contraire : tous les boutons concernent la navigation dans le calendrier (précédent, suivant, année, aujourd'hui) et SEUL le clic sur une date sélectionne une date.

Bref j'ai envoyé #5198 qui rend les choses plus claires

  • au clic sur le bouton on retourne sur le mois qui contient la date du jour
  • laquelle date est surlignée
  • il suffit d'un clic dessus pour la sélectionner

Et encore une fois tout cela est personalisable via les data-x sur les input au cas par cas

Je comprends pas ta phrase @maieul "Le "aujourd'hui" est ambigue, car tout le reste concerne des dates selectionnées, du coup pour moi c'était une selection." Au contraire : tous les boutons concernent la navigation dans le calendrier (précédent, suivant, année, aujourd'hui) et SEUL le clic sur une date sélectionne une date. Bref j'ai envoyé https://git.spip.net/spip/spip/pulls/5198 qui rend les choses plus claires - au clic sur le bouton on retourne sur le mois qui contient la date du jour - laquelle date est surlignée - il suffit d'un clic dessus pour la sélectionner Et encore une fois tout cela est personalisable via les data-x sur les input au cas par cas
maieul commented 10 months ago
Poster
Collaborator

Ah oui, mais les boutons sont au dessus de la grille, alors que le "aujourd'hui" est en en dessous donc pour moi c'est une date sur laquelle on clique pour la choisir, pas un bouton de nav. Si on mettait au dessus.

En gros là on a

  1. Navigation
  2. Selection
  3. Navigation (aujourd'hui)

Moi je comprennais

  1. Navigation
  2. Selection
  3. Selection (aujourd'hui)

Si on pouvait avoir

  1. Navigation
  2. Navigation (aujourd'hui)
  3. Selection il n'y aurait plus d'ambiguité
Ah oui, mais les boutons sont _au dessus_ de la grille, alors que le "aujourd'hui" est en _en dessous_ donc pour moi c'est une date sur laquelle on clique pour la choisir, pas un bouton de nav. Si on mettait au dessus. En gros là on a 1. Navigation 2. Selection 3. Navigation (aujourd'hui) Moi je comprennais 1. Navigation 2. Selection 3. Selection (aujourd'hui) Si on pouvait avoir 1. Navigation 2. Navigation (aujourd'hui) 3. Selection il n'y aurait plus d'ambiguité
cerdic commented 10 months ago
Owner

j'ai bien compris que t'avais pas compris, mais avec #5198 maintenant c'est tout compris. compris ?

j'ai bien compris que t'avais pas compris, mais avec #5198 maintenant c'est tout compris. compris ?
maieul commented 10 months ago
Poster
Collaborator

Disons que cela ne résoud pas tout, mais que c'est deja pas mal, et que donc je propose qu'on ferme, et qu'on se prendra plus la tete si on a d'autres personnes dans le même cas que moi.

Disons que cela ne résoud pas tout, mais que c'est deja pas mal, et que donc je propose qu'on ferme, et qu'on se prendra plus la tete si on a d'autres personnes dans le même cas que moi.
marcimat commented 10 months ago
Owner

C’est intégré.

C’est intégré.
marcimat closed this issue 10 months ago
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
1.8
1.9.1
1.9.2
2.0
2.1
3.0
3.1
3.2
4.0
4.1
4.2
boutons-danger
coquille_doc
debug_ecrire_fichier
dev-sortable
dev/autoloader
dev/hasard_fixe
dev/instituer_ergo
dev/issue_4626_menu_squelettes
dev/issue_5496_intro_ligne_null
dev_infos_image
fix/valider_url_distante
fix_issue_5454
fix_modifier_login
issue_4101
issue_4678
issue_4705
issue_4717
issue_4836
issue_4946
issue_5095
issue_5344
issue_5427_bis
master
refactor_extraire_balises
remove_no_image_filtrer
trigger_deprecated_42
Labels
Apply labels
Clear labels
accessibilité amélioration
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
No Label accessibilité amélioration APIs authentification base de données bug code généré compilo css divers documentation doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide javascript langues LDAP plugin PostgreSQL refusé sécurité traduction
Milestone
Set milestone
Clear milestone
No items
No Milestone
4.0
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(MISSING)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes