Skip to content

Conversation

@pierre56
Copy link
Contributor

   (L.control as any).locate({
      position: 'topright',    // place le bouton en haut à droite
      watch: true,             // active le suivi permanent (navigator.watchPosition)
      setView: 'always',       // recentre la carte à chaque mise à jour
      flyTo: true,             // animation fluide pour les transitions
      showPopup: false         // pas de popup "Vous êtes ici"

Modif afin d'avoir l'option de géoloc disponible même sans activer l'édition pour monitoring,
en plaçant le bouton en haut a droite.

Ajout des options watch -> tracking pour suivre le déplacement
et setview -> utile pour revenir automatiquement a la position gps, même en changeant de fenêtre

exemple concret :
Je finis de saisir une observation sur un site, je reviens au niveau module.
Je suis recentré automatiquement sur ma nouvelle position GPS proche de mon site suivant, ce qui permet de le sélectionner facilement via la carto (pas besoin de taper le nom dans le filtre).

image

       (L.control as any).locate({
          position: 'topright',    // place le bouton en haut à droite
          watch: true,             // active le suivi permanent (navigator.watchPosition)
          setView: 'always',       // recentre la carte à chaque mise à jour
          flyTo: true,             // animation fluide pour les transitions
          showPopup: false         // pas de popup "Vous êtes ici"

  

Modif afin d'avoir l'option de géoloc disponible même sans activer l'édition en plaçant le bouton en haut a droite.
utile pour monitoring
@codecov
Copy link

codecov bot commented Sep 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.48%. Comparing base (8382e12) to head (1307168).
⚠️ Report is 27 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3718      +/-   ##
===========================================
- Coverage    84.54%   84.48%   -0.06%     
===========================================
  Files          133      133              
  Lines        10336    10361      +25     
===========================================
+ Hits          8739     8754      +15     
- Misses        1597     1607      +10     
Flag Coverage Δ
pytest 84.48% <ø> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@jacquesfize jacquesfize left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok pour le déplacement de l'icône, c'est plus cohérent en terme d'usages.

Pour moi le paramètre flyTo doit être mis à false, car ralentit l'utilisateur dans son usage de la geoloc dans GN.

Pour le absence de l'icône de geoloc dans la carte de monitorings, j'ai ouvert une issue PnX-SI/gn_module_monitoring#505.

@jacquesfize jacquesfize added this to the 2.17.0 milestone Sep 30, 2025
@jacquesfize jacquesfize mentioned this pull request Sep 30, 2025
13 tasks
Lors de mes tests, j'ai pas eu trop de lags.
Je trouvais l'animation sympa mais pas de soucis pour que ça dégage :)

Co-authored-by: Jacques Fize <[email protected]>
@camillemonchicourt
Copy link
Member

C'est normal que le bouton n'était pas visible actuellement en mode LISTE, il était uniquement prévu quand on est en saisie d'une observation pour faciliter le fait de se localiser facilement sur sa position si on utilise GeoNature sur le terrain (qui n'est pas prévu pour ça comme déjà évoqué).
Mais OK, pourquoi pas ajouter le bouton aussi sur les carte en mode Consultation / Listes, mais que si on active la fonctionnalité de localisation GPS, qui ne l'est pas par défaut, car on ne peut pas laisser penser actuellement que GeoNature est adaptée à un usage mobile, car cela ne fait pas partie de son périmètre global actuellement.

@jacquesfize
Copy link
Contributor

jacquesfize commented Sep 30, 2025

Mais OK, pourquoi pas ajouter le bouton aussi sur les carte en mode Consultation / Listes, mais que si on active la fonctionnalité de localisation GPS, qui ne l'est pas par défaut, car on ne peut pas laisser penser actuellement que GeoNature est adaptée à un usage mobile, car cela ne fait pas partie de son périmètre global actuellement.

Dans la définition du composant Carte --dans tous les cas-- la géolocalisation est activé uniquement si le paramètre en configuration est indiqué à vrai.

@pierre56
Copy link
Contributor Author

pierre56 commented Sep 30, 2025

car on ne peut pas laisser penser actuellement que GeoNature est adaptée à un usage mobile, car cela ne fait pas partie de son périmètre global actuellement

@camillemonchicourt
J'ai vu passer a plusieurs reprises cette position mais pourquoi ?
C'est utilisé en interne depuis un bon moment via mobile, un peu occtax mais surtout pour monitoring.

J'ai bidouillé un peu le css pour que ce soit plus propre niveau affichage pour téléphone/tablette
mais pas beaucoup au final et il suffirait de pas grand chose en plus pour finir le taff.

Le seul cas où c'est vraiment problématique,
c'est en cas d'absence de réseau où il faut basculer sur l'application qui permet la saisie offline.

@camillemonchicourt
Copy link
Member

Car je pense que bien faire une application web/mobile est une vraie approche globale, technique et fonctionnelle qui demande de la conception, des fonctionnalités, des cas d'usage bien identifiés, etc...
Et GeoNature a été conçu et développé pour un usage web desktop. Pour le mobile, on s'est concentré sur Occtax-mobile et désormais Monitoring-mobile.

Je ne dis pas que cela ne fonctionne pas avec des ajustements et "bidouilles", mais ce n'est pas prévu globalement pour et on n'a pas la capacité de lancer un tel sujet actuellement et de le supporter.
Mais vu que ça fonctionne à peu prêt pour ceux qui veulent, on a intégré l'ajout de la géolocalisation sans l'activer par défaut, pour ceux qui veulent mais sans assumer/supporter que GeoNature est globalement adapté et fonctionnel en mobile.

jacquesfize added a commit that referenced this pull request Oct 13, 2025
@jacquesfize jacquesfize mentioned this pull request Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants