Viglino / Map-georeferencer

A proof of concept to georeference maps with Openlayers (ol).
http://viglino.github.io/Map-georeferencer/
65 stars 17 forks source link

améliorations et bug #4

Open EPINRAP opened 5 years ago

EPINRAP commented 5 years ago

Bonjour et bravo pour cet utilitaire. J'aimerai pouvoir l'intégrer dans mon interface et l'améliorer :

Des suggestions ? Amicalement Eric

Viglino commented 5 years ago

Bonjour, Il y aurait effectivement pas mal de choses à faire pour améliorer l'application. Je n'ai pas le temps actuellement de m'y consacrer pleinement. La suppression des points est une première étape (deleteCondition ne vaut que pour des points intermédiaires sur une ligne) et demande de gérer le point lié (et le recalcul de la transformation). L'ajout de point via des coordonnées serait également une amélioration mais demande de gérer des projections pour être pleinement utilisable). @+

EPINRAP commented 4 years ago

salut à toutes

fins utiles la version 6 d'OL a supprimé le ol.inherits j'ai rajouté une fonction ol_inherits : function ol_inherits(child,parent) { child.prototype = Object.create(parent.prototype); child.prototype.constructor = child; }, après il suffit de remplacer ol.inherits par ol_inherits dans le ol-ext.min.js et le geoportail.js

Je me remets sur le georef pour tenter de l'améliorer, mais comme je suis novice, ça prends beaucoup de temps

Viglino commented 4 years ago

ol-ext a sa propre fonction ol.ext.inherits pour assurer la compatibilité avec ol6.

EPINRAP commented 4 years ago

ok cette fonction n'est pas implémentée dans l'exemple http://viglino.github.io/Map-georeferencer/ et c'est pour rendre service que je partageais la modification car j'ai du modifier le code moi même pour qu'il fonctionne en ol6

Viglino commented 4 years ago

Pas de problème. En fait il faudrait mettre à jour la dépendance pour y avoir accès. La dernière version d'ol-ext est compatible avec ol6. Ensuite il y a quelques modification à apporter dans le code du projet pour qu'il soit compatible avec ol6. @+