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

Closed
opened 3 months ago by maieul · 12 comments
maieul commented 3 months ago

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 3 months ago
Owner

Version cible ?

Version cible ?
Poster

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 3 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 ?
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.
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.
Poster

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
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
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
Poster

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é
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 ?
Poster

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.
Owner

C’est intégré.

C’est intégré.
marcimat closed this issue 3 months ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.