Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Occtax : comportement de la carte (zoom) #912

Closed
sylvain-m opened this issue May 4, 2020 · 7 comments
Closed

Occtax : comportement de la carte (zoom) #912

sylvain-m opened this issue May 4, 2020 · 7 comments

Comments

@sylvain-m
Copy link
Contributor

Bonjour à tous,

Excusez moi si cela a déjà été discuté (j'ai trouvé l'issue #570 en lien, mais ça commence à dater).
Je commence à saisir "en conditions réelles" des données dans OccTax.
Lorsque j'ouvre OccTax, j'ai tendance à zoomer directement sur le lieu de ma saisie, pour faire un premier repérage (aussi parce-que, instinctivement, j'oublie que je ne suis pas encore dans le formulaire de saisie du relevé).
Sauf que, quand je clique sur "ajouter un relevé", je perds tout le zoom que j'avais fait pour revenir au zoom initial : dommage de devoir re-géolocaliser notre relevé ensuite.
S'il était possible de conserver la bbox de la carte au moment de l'ouverture du relevé, ce serait top !

Par ailleurs, j'ai constaté que, lorsqu'on pointe un relevé, et uniquement en mode "point" (pas pour les lignes ou les polygones), la carte dézoome automatiquement : personnellement, je trouve ce comportement inutile, et même perturbant (on se demande si on a cliqué au bon endroit, parce qu’une fois dézoomé, on ne voit pas précisément où a été placé le point).

Je peux regarder plus précisément si j'identifie les lignes de codes responsables, mais avant, je veux bien savoir si vous avez déjà discuté de ça, ou si vous partagez mes 2 suggestions :

  • conserver le zoom entre ouverture Occtax et saisie relevé
  • annuler le "dé-zoom" automatique à la saisie d'un point

A+

Sylvain

@sylvain-m
Copy link
Contributor Author

sylvain-m commented May 4, 2020

Je pense que le problème du "dé-zoom" vient de là :

/* Boolean qui controle le zoom au point*/
@Input() bZoomOnPoint = true;
@Input() zoomLevelOnPoint = 8;

Du coup, on se rend compte que c'était sensé être un zoom vers le niveau 8, mais lorsqu'on est déjà à plus de 8, ça se transforme en "dé-zoom".
Je proposerais bien de mettre false pour ce paramètre, ou alors au contraire, de mettre 18 pour le niveau après saisie.

@samuelpriou
Copy link

samuelpriou commented May 4, 2020

Bonjour à tous,

Je partage le point de vue de Sylvain.
Lorsque j'ouvre OccTax, j'ai moi aussi tendance à :

"zoomer directement sur le lieu de ma saisie, pour faire un premier repérage (aussi parce-que, instinctivement, j'oublie que je ne suis pas encore dans le formulaire de saisie du relevé).
Sauf que, quand je clique sur "ajouter un relevé", je perds tout le zoom que j'avais fait pour revenir au zoom initial : dommage de devoir re-géolocaliser notre relevé ensuite."

Je suis également d'avis de conserver la bbox de la carte au moment de l'ouverture du relevé !

@jbrieuclp
Copy link
Contributor

Ça se règle avec un énième service Angular, mais lui général au module occtax.

@sylvain-m
Copy link
Contributor Author

Ça se règle avec un énième service Angular, mais lui général au module occtax.

Je suppose que ça veut dire que c'est pas compliqué 😉

@camillemonchicourt
Copy link
Member

Doublon avec #436

@sylvain-m
Copy link
Contributor Author

Pour ce qui est du niveau de zoom après pointage du relevé, j'ai tenté une PR :
#932
(en espérant l'avoir fait "dans les règles", car je ne sais pas faire de PR 😉 )

@camillemonchicourt
Copy link
Member

Réglé dans la 2.5.0 où l'on conserve le zoom de la page Carte/Liste quand on passe sur la création d'un nouveau relevé.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants