Projet

Général

Profil

Bug #14736

Affichage d'une journée (31 Décembre 1899) non nécessaire

Ajouté par Brice Mallet il y a environ 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
23 janvier 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

A la "Modification de la période horaire" (https://chrono-alfortville.test.entrouvert.org/manage/timeperiods/14/edit et copie d'écran jointe) lorsque l'on édite les bornes horaires pour une journée de la semaine, en titre du pop-up s'affiche une date : "31 Décembre 1899".
Celle-ci n'est pas nécessaire puisqu'il s'agit de choisir une heure uniquement, et non l'heure d'une date, dans ce champ.
Demande : ne pas afficher de date sélectionnable dans cette aide à la saisie d'une heure


Fichiers

Révisions associées

Révision b181bd8d (diff)
Ajouté par Frédéric Péters il y a presque 7 ans

hide header from time-only picker (#14736)

Historique

#1

Mis à jour par Brice Mallet il y a environ 7 ans

Plus globalement, je trouve surprenant ce mode de sélection d'une heure avec en :
1er : une heure pile de la journée
2ème : cette heure avec minutes (de x:00 à x:55)
Ne serait-il pas plus compréhensible de proposer un premier choix des heures puis un deuxième choix des minutes uniquement, sans rappel de l'heure ?

#2

Mis à jour par Frédéric Péters il y a environ 7 ans

La seule évolution envisagée de mon côté c'est virer le widget et taper un <input type=time>, ça gênera juste un peu pour Firefox pendant quelques mois mais avec de toute façon un petit mot gentil sous le champ ça doit aller.

#3

Mis à jour par Mikaël Ates il y a presque 7 ans

Petit up sur le sujet. Je confirme que c'est perturbant de pouvoir sélectionner une date en même temps que l'heure alors que seule l'heure est nécessaire, cela laisse penser, lors de la prise en main, qu'il est possible de limiter avec des dates la période horaire.

#4

Mis à jour par Frédéric Péters il y a presque 7 ans

Ça affiche mais ça ne laisse pas sélectionner de date, ou alors faut filer une URL.

#5

Mis à jour par Thomas Noël il y a presque 7 ans

J'ai pas compris le "ou alors faut filer une URL", mais ack pour le patch.

#6

Mis à jour par Frédéric Péters il y a presque 7 ans

  • Statut changé de En cours à Résolu (à déployer)

C'est parce que je n'ai pas compris le "pouvoir sélectionner une date", du coup je demandais une URL où constater ça.

commit b181bd8d7862f410b1e20d505c0ecd6ee676646e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Apr 11 10:16:32 2017 +0200

    hide header from time-only picker (#14736)
#7

Mis à jour par Mikaël Ates il y a presque 7 ans

https://agendas-test-entrouvert.montpellier3m.fr/manage/timeperiods/1/edit, on peut "sélectionner" un jour avec les flèches du widget.

#8

Mis à jour par Frédéric Péters il y a presque 7 ans

Ok j'avais zappé les flèches, elles disparaissent avec ce commit.

#9

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF