Gestion des cas où il n'y a pas d'horaires fixes

esmenard

Le but ce cette PR est d'afficher correctement les horaires d'ouverture de la forme opening_hours="<un commentaire qui décrit quand c'est ouvert>", utilisé pour les lieux qui peuvent être ouverts ou fermés mais pas à heure fixe. En reprenant l'exemple de #537, voilà ce que ça donne :

Avant

Après

Les choses qui manquent selon moi :

• Chosir ce qu'on affiche comme pictogramme à gauche des horaires : est-ce qu'on laisse le picto fermé, ou est-ce qu'on choisit un picto spécifique ?
• Masquer le menu déroulant en dessous, qui affiche toujours les mêmes bêtises (à savoir que c'est ouvert de 0h à 0h aujourd'hui et fermé les autres jours). Et en profiter pour le masquer aussi quand on a opening_hours=24/7, "Ouvert tous les jours à toute heure", n'apporte pas d'information supplémantaire par rapport à "Ouvert 24h/24 7j/7".
• Gérer séparément le cas de opening_hours=unknown (834 occurences d'après Taginfo), pour lesquels ça ne sert à rien d'afficher les horaires puisqu'il n'y a rien à en tirer. Dans la version actuelle de la PR ils affichent "Pas d'horaires fixes :"
• Dans l'écran de modification des horaires afficher comme si les horaires n'étaient pas renseignés si opening_hours=unknown

Fix #537

1. esmenard

   J'ai ajouté le fait de masquer les horaires d'ouverture pour les lieux avec opening_hours=unknown. Par exemple sur https://cartes.app/?allez=d%25C3%25A9fibrillateur%7Cn9273391901%7C2.23326%7C48.71936

   Avant

   Après

2. esmenard

   Les détails des horaires d'ouverture sont maintenant masqués quand l'état d'ouverture ne varie pas en fonction du temps (c'est-à-dire dans les cas opening_hours=closed, opening_hours=24/7 et opening_hours="<un commentaire>", le cas opening_hours=unknown est traité séparément, cf. https://codeberg.org/cartes/web/pulls/2819#issuecomment-21937472), sauf en mode modification des horaires. Voilà ce que ça donne pour chaque cas de figure :

   • opening_hours=24/7 (https://cartes.app/?allez=Parking+v%25C3%25A9lo+s%25C3%25A9curis%25C3%25A9%7Cw1313106522%7C2.23720%7C48.70749)

   Avant Après

   • opening_hours="<un commentaire>" (https://cartes.app/?allez=Isa%2527Belles+Patounes%7Cw1319669202%7C2.23307%7C48.70162)

   Avant Après

   • opening_hours=closed (https://cartes.app/?allez=M%25C3%25A9diath%25C3%25A8que+George+Sand%7Cw71346704%7C2.24232%7C48.71165)

   Avant Après

   Au passage ça corrige un autre bug : il n'est pas possible de modifier les horaires d'un lieu avec opening_hours=24/7. Sur l'exemple montré plus tôt, voilà ce que ça donne.

   Avant Après

3. esmenard

   @esmenard wrote in https://codeberg.org/cartes/web/pulls/2819#issue-6928106:

   • Dans l'écran de modification des horaires afficher comme si les horaires n'étaient pas renseignés si opening_hours=unknown

   Corrigé en incorporant les modifications de #2951, à la place du "ouvert de 0h à 0h aujourd'hui et fermé les autres jours" on a un bouton pour réinitialiser les horaires.

   Je vais soumettre en l'état même si c'est pas parfait (notamment quand on passe sur des horaires pas déroulables le curseur change quand même de tête comme si ils étaient déroulables), les problèmes qui restent pourront être corrigés après.