Projet

Général

Profil

Development #15533

Sur le rendu carte des demandes géolocalisées attribuer aux epingles les couleurs des status

Ajouté par Serghei Mihai il y a environ 7 ans. Mis à jour il y a presque 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
21 mars 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour que les agents puissent distinguer les zones ou ils doivent agir.


Fichiers


Demandes liées

Lié à w.c.s. - Development #15532: popup avec les infos de la demande sur clic d'une point sur une carte des demandes géolocaliséesFermé21 mars 2017

Actions

Révisions associées

Révision 2411c70f (diff)
Ajouté par Serghei Mihai il y a presque 7 ans

backoffice: display status-coloured markers on maps (#15533)

Historique

#2

Mis à jour par Serghei Mihai il y a presque 7 ans

Je propose d'utiliser les markers circles à la place du marker par défaut (qui est une image).

#3

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

  • Fichier Capture d’écran 2017-04-10 à 19.54.57.png ajouté
  • Fichier Capture d’écran 2017-04-10 à 19.54.43.png ajouté

Les deux autres endroits du code qui récupèrent la couleur d'un statut le fond dans un try/except AttributeError et celui dans wcs/forms/common.py est visité lors des tests, je me dis donc que ça peut arriver.

Sur le style c'est quand même moins marquant que l'actuel, peut-être augmenter l'opacity, jusque 0.9, et doubler la largeur de la bordure ?

Côté code, vu que le code commun s'allonge, factoriser un peu ? (en fait c'est très facile, suffit de déclarer le on('backoffice-filter-change') en premier et de faire un trigger de ce signal.)

#4

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

  • Fichier Capture d’écran 2017-04-10 à 19.54.57.png supprimé
#5

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

  • Fichier Capture d’écran 2017-04-10 à 19.54.43.png supprimé
#7

Mis à jour par Pierre Cros il y a presque 7 ans

C'est peut-être lié aux tuiles de la carte utilisées sur les captures, mais je trouve que les circles ça fait un peu vieux schnock, désolé :/. Si c'est facile d'avoir autre chose tant mieux sinon tant pis.

#8

Mis à jour par Serghei Mihai il y a presque 7 ans

Je suis parti du fonctionnel, ou le marquer doit avoir la couleur du statut.
Et le moyen facile embarqué dans leaflet est le marqueur circle. Sinon il faut générer des images (comme le marqueur bleu actuel) avec les couleurs de chaque statut possible du workflow.
Je vais regarder ce qu'il est possible de faire.

#9

Mis à jour par Serghei Mihai il y a presque 7 ans

Et comme ça, ça paraît plus moderne ?

#10

Mis à jour par Pierre Cros il y a presque 7 ans

Oui, bien plus, merci Serghei.

#11

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

Utiliser L.divIcon + le marqueur de FontAwesome http://fontawesome.io/icon/map-marker/ ?

#12

Mis à jour par Pierre Cros il y a presque 7 ans

C'est encore mieux c'est clair.

#13

Mis à jour par Serghei Mihai il y a presque 7 ans

Je préfère un divIcon dans lequel construire un element dessinant le marker (ma capture précedente).
La raison pour cela est que dans la future cellule du portail agent ou les markers contiendront l'icone du type de la demarche, l'affichage (et le code) sera le même, avec l'icone au milieu.

#14

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

Dans l'idée alors, léger ajustement au style quand même, avec :

    border-radius: 11rem 6rem 0.8rem;
    transform: scale(1, 1.3) rotate(45deg);
    border: 1px solid #888;

Aussi ça serait bien de taper ce bout de style en fin de fichier, là où il y a déjà backoffice-map, et puis d'indenter avec des tabulations vu que c'est presque utilisé partout dans le fichier.

Et dans le message de commit, parler de formdata plutôt que de demands, ou juste "backoffice: display status-coloured markers on maps".

Tester aussi la carte globale (test_global_listing_geojson).

#16

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

  • Lié à Development #15532: popup avec les infos de la demande sur clic d'une point sur une carte des demandes géolocalisées ajouté
#17

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

Dans le patch dans #15532 tu modifies l'appel .trigger() pour passer un autre paramètre, c'est vraiment lié à l'autre patch ou c'est une correction repérée par rapport à celui-ci ?

#18

Mis à jour par Serghei Mihai il y a presque 7 ans

C'est lié à l'autre patch. Ici je passe le formulaire serializé car c'est un parametre attendu par la fonction.
Et dans celui de #15532 je lui passe les parametres GET afin d'avoir la liste des champs des formulaires à afficher.

#19

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

Et dans celui de #15532 je lui passe les parametres GET afin d'avoir la liste des champs des formulaires à afficher.

Ok, j'ai compris ça en lisant et testant, et la modification de l'autre côté doit sans doute disparaitre parce que « actuellement, sur une modification de filtre, les champs seront réinitialisés. ».

#20

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

Bon, ack, et on pourra encore faire évoluer un peu les marqueurs, genre mettre de l'ombre, des trucs comme ça, par la suite.

#21

Mis à jour par Serghei Mihai il y a presque 7 ans

  • Statut changé de Nouveau à Résolu (à déployer)
commit 2411c70fedbacbfde9b21cb45247cf3e6ada24b1
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Mon Apr 10 19:12:26 2017 +0200

    backoffice: display status-coloured markers on maps (#15533)
#22

Mis à jour par Serghei Mihai il y a presque 7 ans

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

Formats disponibles : Atom PDF