search

Geoportail-Luxembourg / geoportailv3

geoportailv3 is the implementation of the v3 of the map viewer of the luxembourgish geoportal
MIT License
25 stars 16 forks source link

Donnée selectionnée / Ordre des couches [c2c, lux] #217

Closed sbrunner closed 9 years ago

sbrunner commented 9 years ago

Get from #32 Restant c2c 01/01/2015 2j Restant c2c 01/02/2015 fini

elemoine commented 9 years ago

Voir ce commentaire pour le travail d'intégration : https://github.com/Geoportail-Luxembourg/geoportailv3/issues/32#issuecomment-72057153.

jaykayone commented 9 years ago

A faire par @rmichaelis

pgiraud commented 9 years ago

@elemoine doit d'abord finaliser un patch pour supporter les touch events.

elemoine commented 9 years ago

Voici l'issue ngeo : https://github.com/camptocamp/ngeo/issues/154.

elemoine commented 9 years ago

Comme indiqué dans l'issue ngeo j'ai noté un bug dans Closure Library. J'ai fait un PR pour Closure Library mais il n'est pas encore mergé. Nous sommes donc en attente.

jaykayone commented 9 years ago

si je comprends bien, ce patch concerne le touch. nous pouvons donc déjà implémenter la fonctionnalité côté desktop et attendre que ta PR soit mergée?

elemoine commented 9 years ago

Oui, je suis d'accord. Un développeur de Closure Library m'a conseillé de mieux expliquer le problème dans ma PR. Je vais tenter de faire ça cette semaine.

elemoine commented 9 years ago

Je suis acutellement assigné à cette tâche. Faut-il réassigner ?

jaykayone commented 9 years ago

A toi de me dire :) si pour toi tout est prêt, on la passe à @rmichaelis

elemoine commented 9 years ago

En fait il faudrait qu'on merge ma branche ngeo avant. Je vous tiens au courant.

elemoine commented 9 years ago

Ok, le PR ngeo a été mergé. C'est l'exemple layerorder qu'il faut regarder.

elemoine commented 9 years ago

Oui, je suis d'accord. Un développeur de Closure Library m'a conseillé de mieux expliquer le problème dans ma PR. Je vais tenter de faire ça cette semaine.

Pour info, j'ai mis à jour ma PR à closure-library pour bien expliquer le problème et la correction, mais je n'ai pas de nouvelle.

jaykayone commented 9 years ago

OK, à faire par @petzlux

elemoine commented 9 years ago

À noter l'utilisation du service ngeoSyncArrays dans l'exemple layerorder. Il faut utiliser ce service pour le "re-ordering" des layers.

petzlux commented 9 years ago

336

petzlux commented 9 years ago

After some styling adjustments by @pgiraud , can this be considered done ? I quite like that there is some contrast and simplification in styling between dragged element and other elements ...

pgiraud commented 9 years ago

Yes. This is still not perfect on mobile, but it can be addressed in a different issue.