mviewer icon indicating copy to clipboard operation
mviewer copied to clipboard

Time calendar - afficher les dates disponibles

Open Gaetanbrl opened this issue 2 years ago • 7 comments

ref #870

Cette PR permet de mettre en surbrillance les dates disponibles (publiées par la dimension temps) et permet de ne sélectionner que celles-ci.

Pour que ce mode de visualisation soit optionnel, un nouveau paramètre timeshowavailable a été ajouté.

image

Contenu :

  • [x] code
  • [x] documentation

Gaetanbrl avatar Mar 28 '24 15:03 Gaetanbrl

quelqu'un pour tester cette PR svp ? Sinon je merge :)

Gaetanbrl avatar Jul 05 '24 06:07 Gaetanbrl

J'ai parlé trop vite.

J'ai un souci. Version actuelle : https://kartenn.region-bretagne.fr/kartoviz/?x=-220750&y=6144926&z=8&l=demandeur_emploi_geomdbl3**level%20%3D%20%27EPCI%27*2024-03-01&lb=osm_grey&config=apps/region/territoire/territoire.xml&mode=d

Version en test : https://kartenn.region-bretagne.fr/kartoviz-jdev/?x=-220750&y=6144926&z=8&l=demandeur_emploi_geomdbl1**level%20%3D%20%27EPCI%27%2Cepci&lb=osm_grey&config=apps/region/territoire/territoire.xml&mode=d#

en cliquant sur le calendrier, je ne peux pas changer d'année. Je suis en affichage uniquement du mois

lecault avatar Jul 05 '24 10:07 lecault

Extrait du XML : timefilter="true" timeinterval="month" timecontrol="calendar" timemin="2017-02-01" timemax="2024-12-01" timeshowavailable="true"

lecault avatar Jul 05 '24 11:07 lecault

Ok je regarde merci @lecault

Gaetanbrl avatar Jul 08 '24 14:07 Gaetanbrl

en cliquant sur le calendrier, je ne peux pas changer d'année. Je suis en affichage uniquement du mois

Ok, c'est assez perturbant d'afficher des jours quand on a un pas de temps au mois, mais en effet je pense que je dois vérifier la prise en compte des pas de temps autres qu'en date (mois / année).

Gaetanbrl avatar Jul 08 '24 14:07 Gaetanbrl

Le problème provient potentiellement de l'event de comparaison ici : https://github.com/mviewer/mviewer/pull/881/files#diff-fcd3df1e9382e80da53677b97af464ae3c143dad8052280ebff0fe28bc95fac4R2919

... je poursuis l'analyse

Gaetanbrl avatar Jul 08 '24 15:07 Gaetanbrl

@lecault est-ce possible de retester sur ton environnement suite à mon dernier commit ?

Gaetanbrl avatar Jul 09 '24 09:07 Gaetanbrl

J'ai toujours le souci...

lecault avatar Jul 18 '24 07:07 lecault

J'ai toujours le souci...

Je ne suis pas certain de comprendre ce que tu as comme problème alors.

Peux-tu me mettre si possible :

  1. la config que tu utilises + la couche
  2. un gif de ce que tu vois
  3. ce que tu devrai voir

merci :)

Gaetanbrl avatar Jul 18 '24 07:07 Gaetanbrl

Ah en fait ça marche, c'est mon permalien qui était pas bon.

https://kartenn.region-bretagne.fr/kartoviz-jdev/?x=-220750&y=6144926&z=8&l=demandeur_emploi_geomdbl3**level%20%3D%20%27EPCI%27&lb=osm_grey&config=apps/region/territoire/territoire.xml&mode=d#

Et c'est bien pour nous comme évolution merci.

date_jdev

lecault avatar Jul 22 '24 14:07 lecault

Ah en fait ça marche, c'est mon permalien qui était pas bon.

Super ! :)

On Merge ?

Gaetanbrl avatar Jul 23 '24 07:07 Gaetanbrl

Yes !

lecault avatar Jul 23 '24 08:07 lecault

Merci @lecault

Gaetanbrl avatar Jul 23 '24 08:07 Gaetanbrl

une évolution que j'ai oublié serait d'augmenter le contraste entre les valeurs valides et les autres car dans mon exemple, l'écart n'est pas très visible.

lecault avatar Jul 23 '24 09:07 lecault

Je pense qu'avec le theme, ca doit être facilement modifiable :

https://github.com/mviewer/mviewer/pull/881/files#diff-ef9fb8ce54d505012ea5e07c9dc058e41f07242c8065f75cf5d865d95099e9baR2032

Gaetanbrl avatar Jul 23 '24 09:07 Gaetanbrl