Ajout de l'edition des coordonnées #117

pprev94 · 2017-05-24T08:58:38Z

Possibilité de pouvoir effectuer une recherche via la nouvelle option editCoordinates (false par défaut)

…on.js et messages d'erreurs en anglais

gcebelieu · 2017-05-29T09:08:55Z

@pprev94 : premier retour, il faudrait résoudre les erreurs de style de codage (erreurs jscs) qui empêchent la compilation travis de passer :

https://travis-ci.org/IGNF/geoportal-extensions/builds/235557792?utm_source=github_status&utm_medium=notification

Tu as accès aux logs ?

gcebelieu

Une remarque de fond : je préfèrerais que la saisie des coordonnées se fasse directement dans le panneau d'affichage des coordonnées actuel (qu'il faudrait modifier pour l'occasion) plutôt que dans un nouveau panneau : cela complique l'ergonomie du widget.
une remarque sur la validation de la saisie : la gestion du séparateur "," non autorisé est perturbante car elle positionne l'internaute à un endroit non voulu sans rien dire
un détail : attention aux indentations (même dans les exemples) avec des tabulations : elles ne sont pas autorisées dans ce projet utiliser des indentations à quatre espaces à la place.

gcebelieu · 2017-05-29T13:59:42Z

samples/ol3/MousePosition/amd-all-options.html

@@ -78,6 +78,7 @@

                        var mp = new MousePosition({
                            collapsed : false,
+							              editCoordinates : true,


attention : ne pas utiliser de tabulations pour l'indentation !

gcebelieu · 2017-05-29T14:23:09Z

src/Common/Controls/MousePositionDOM.js

@@ -113,6 +113,9 @@ define([], function () {
            div.className = "GPpanel";

            div.appendChild(this._createMousePositionPanelHeaderElement());
+            if (this.options.editCoordinates) {
+                div.appendChild(this._createEditCoordinatesPanelElement());


Pour moi, la saisie des coordonnées doit se faire dans le même panneau que celui de l'affichage des coordonnées.
=> ne pas créer de nouveau panneau mais réutiliser celui de l'affichage en modifiant les types d'input qui les affichent. Cf. fonction _createMousePositionPanelBasicCoordinateElement(), à laquelle on pourrait passer l'option "editCoordinates" en paramètre

gcebelieu · 2017-05-29T14:31:26Z

src/Common/Controls/MousePositionDOM.js

+        },
+
+        /** ... */
+        _createBasicEditCoordinatesElement : function (coordType) {


Attention à la validation de la saisie : lorsqu'on saisit une valeur avec une "," au lieu d'un "." comme séparateur, le widget ne tient pas compte de la partie après la virgule mais n'affiche aucun message ... Il faudrait : soit empêcher le positionnement en indiquant que la saisie n'est pas bonne, soit changer à la volée, la "," en ".".

gcebelieu · 2017-06-09T12:40:25Z

ça ne semble pas très bien marcher :

En testant sur l'exemple MousePosition/amd-all-options.html

l'affichage des input est décalé par rapport aux labels :

Il faudrait qu'on ait la valeur à la suite du label (comme avant) :
"x : 455544"
"y : 987878"

Il faudrait aussi que les contours des inputs ne soient pas visibles (comme avant) : ce n'est pas bloquant, on peut régler cela dans un second temps.

ensuite lorsqu'on change de système de coordonnées (passage en géographique par exemple), l'affichage ne se fait plus :

Et on a une erreur dans la console :

14:28:58,406 TypeError: elLat is null 1 MousePositionDOM.js:622:21
	MousePositionDOM.GPdisplayCoords http://localhost/gilles/apiv3/geoportal-extensions/src/Common/Controls/MousePositionDOM.js:622:21
	MousePosition.prototype._setCoordinate http://localhost/gilles/apiv3/geoportal-extensions/src/Ol3/Controls/MousePosition.js:1004:9
	MousePosition.prototype.onMouseMove http://localhost/gilles/apiv3/geoportal-extensions/src/Ol3/Controls/MousePosition.js:1058:9
	ol.events.bindListener_/boundListener http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:6671:12
	ol.events.EventTarget.prototype.dispatchEvent http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:7091:11
	ol.Map.prototype.handleMapBrowserEvent http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:31494:7
	ol.events.bindListener_/boundListener http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:6671:12
	ol.events.EventTarget.prototype.dispatchEvent http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:7091:11
	ol.MapBrowserEventHandler.prototype.relayEvent_ http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:12471:3
	ol.events.bindListener_/boundListener http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:6671:12
	ol.events.EventTarget.prototype.dispatchEvent http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:7091:11
	ol.pointer.PointerEventHandler.prototype.fireEvent http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:12122:3
	ol.pointer.PointerEventHandler.prototype.move http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:11990:3
	ol.pointer.MouseSource.prototype.mousemove http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:10630:5
	bound  self-hosted:915:17
	ol.pointer.PointerEventHandler.prototype.eventHandler_ http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:11922:5
	ol.events.bindListener_/boundListener http://localhost/gilles/apiv3/geoportal-extensions/lib/ol3/ol-debug.js:6671:12

gcebelieu · 2017-06-12T15:42:32Z

Le dernier commit corrige les PBs signalés 👍

C'est OK pour cette première partie (affichage dans des inputs). Il reste juste cette remarque non bloquante :

Il faudrait aussi que les contours des inputs ne soient pas visibles (comme avant)

gcebelieu · 2017-06-22T09:04:59Z

ça marche plutôt bien 👍

3 remarques cependant :

Il faudrait que l'utilisateur puisse voir qu'on peut passer en mode édition : en rajoutant des infos qui s'affichent au survol des input ("Cliquer pour saisir des coordonnées"). On pourrait aussi rajouter le bouton de centrage dès le début quand on est en mode affichage qui permettrait d'activer le mode édition.
lorsqu'on est en mode édition et que l'on change les systèmes de coordonnées (par exemple passage de mercator à géographique), les champs de saisie ne se mettent pas à jour et on se retrouve à saisir des lat/long dans des champs X/Y
Validation de la saisie des coordonnées :

Lorsqu'on saisit des coordonnées alors qu'il y a une BBOX de restriction, j'ai l'erreur suivante dans la console :

10:53:43,184 TypeError: extent.getTopLeft is not a function 1 MousePosition.js:1393:29
	MousePosition.prototype.locateCoordinates http://localhost/gilles/apiv3/geoportal-extensions/src/Ol3/Controls/MousePosition.js:1393:29
	MousePosition.prototype.locate http://localhost/gilles/apiv3/geoportal-extensions/src/Ol3/Controls/MousePosition.js:1417:13
	_createMousePositionPanelEditToolsElement/< http://localhost/gilles/apiv3/geoportal-extensions/src/Common/Controls/MousePositionDOM.js:437:17

CSS Commune entre ol et leaflet...

gcebelieu · 2017-06-23T13:14:54Z

NB : sur la dernière version, il y a une inversion de la saisie latitude <-> longitude lorsqu'on est en dégrés décimaux (en sexagésimaux, c'est OK).

(Edition des coordonnées)

into pull-request-117

gcebelieu · 2017-06-29T09:29:21Z

Remarques sur le paramétrage du marker :

dans la doc : la propriété permettant de passer l'url du marker est documentée comme "src" alors que c'est "url" qui est utilisée => il faut modifier la doc pour indiquer la bonne propriété.
offset : il faudrait que les valeurs par défaut de l'offset permettent de positionner correctement le marker par défaut par rapport à la position. Pour cela, les valeurs à appliquer sont : [-25.5, -38].
Pour documenter cette option offset, il faudrait mettre le texte suivant :

Offsets in pixels used when positioning the marker towards targeted point. The first element in the array is the horizontal offset. A positive value shifts the marker right. The second element in the array is the vertical offset. A positive value shifts the marker down. [0,0] value positions the top-left corner of the marker image to the targeted point. Default is offset associated to default marker image.

pprev94 added 2 commits May 24, 2017 10:50

Ajout de l'edition des coordonnées

cae9881

gulpfile.js comme avant

818e4ae

gcebelieu requested a review from lboulanger May 24, 2017 09:18

pprev94 added 2 commits May 24, 2017 11:32

conversion DMS

dfa999b

menage

007be3d

gcebelieu requested review from gcebelieu and removed request for lboulanger May 24, 2017 13:34

ajout de editCoordinates dans le commentaire de la classe MousePoisti…

fff951b

…on.js et messages d'erreurs en anglais

lowzonenose self-requested a review May 29, 2017 09:03

check jshint/jscs for PR#117...

1f7bd62

lowzonenose approved these changes May 29, 2017

View reviewed changes

gcebelieu requested changes May 29, 2017

View reviewed changes

pprev94 added 11 commits June 1, 2017 14:41

edit coordinates nlle mouture

2d139ef

edit coordinates nlle mouture

bba79e5

bricoles

55445cb

refactoring

0375330

ajout de commentaires et nettoyage

6a68d92

correction de bug

ea562f0

correction d'un petit bug

393d115

corrections suite check

f50cc9c

suite check errors

46cec90

check dans gulpfile

0a43e66

correction de bug

5085b2f

css suite pull request gilles

b627706

ajout de l'edition des coordonnees

c6c99ad

suite demandes correction par Gilles

a03e9d4

lowzonenose and others added 2 commits June 23, 2017 10:51

MousePosition Leaflet fonctionnel +

f3f2036

CSS Commune entre ol et leaflet...

suite jp

da57fb5

lowzonenose and others added 6 commits June 23, 2017 16:28

Mise en place d'une version MousePosition Leaflet

81bbc89

(Edition des coordonnées)

ajout des options du marker et de l'overlay

d6a2e76

suite check

7454e7d

MousePosition sous Leaflet, fonctionel...

ccc5242

Merge branch 'master' of https://github.com/pprev94/geoportal-extensions

e8e604f

into pull-request-117

Quelques corrections suite refactoring JPB

695fd7b

suite remarques Gilles doc et offset du marker

10e390f

gcebelieu approved these changes Jun 29, 2017

View reviewed changes

gcebelieu merged commit d24251f into IGNF:master Jun 29, 2017

This was referenced Jul 21, 2017

fix editCoordinates default value to false on vg control #126

Merged

Upgrade SDK MP(editCoordinates) + filter feature properties IGNF/geoportal-sdk#16

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ajout de l'edition des coordonnées #117

Ajout de l'edition des coordonnées #117

pprev94 commented May 24, 2017

gcebelieu commented May 29, 2017

gcebelieu left a comment

gcebelieu May 29, 2017

gcebelieu May 29, 2017

gcebelieu May 29, 2017

gcebelieu commented Jun 9, 2017

gcebelieu commented Jun 12, 2017

gcebelieu commented Jun 22, 2017 •

edited

Loading

gcebelieu commented Jun 23, 2017

gcebelieu commented Jun 29, 2017

Ajout de l'edition des coordonnées #117

Ajout de l'edition des coordonnées #117

Conversation

pprev94 commented May 24, 2017

gcebelieu commented May 29, 2017

gcebelieu left a comment

Choose a reason for hiding this comment

gcebelieu May 29, 2017

Choose a reason for hiding this comment

gcebelieu May 29, 2017

Choose a reason for hiding this comment

gcebelieu May 29, 2017

Choose a reason for hiding this comment

gcebelieu commented Jun 9, 2017

gcebelieu commented Jun 12, 2017

gcebelieu commented Jun 22, 2017 • edited Loading

gcebelieu commented Jun 23, 2017

gcebelieu commented Jun 29, 2017

gcebelieu commented Jun 22, 2017 •

edited

Loading