Bonjour à vous
Sur une installation fraiche, lorsqu’on a défini l’emplacement de notre maison + une ou plusieurs autres zones, et qu’on ouvre la map par la suite, celle-ci s’affiche avec un zoom minimal:
Ici, j’aurais trouvé pratique que le zoom s’effectue de sorte à afficher mes points le plus distinctement possible:
Est-ce voulu ou est-ce un oublie ?
Dans le premier cas, je serais très heureux de connaître la justification derrière ce choix.
Bien à vous,
1 « J'aime »
Salut @GziAzman et bienvenue sur Gladys
Il me semble que ça devrait faire comme tu décris… (zoomer sur les points créé)
Carrément je veux bien une issue et une PR
Merci !
@pierre-gilles Merci pour ta réponse.
GladysAssistant:master ← GziAzman:fix-map-initial-bounding
opened 09:01AM - 18 Mar 24 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [ ] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [ ] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
CF #2041
**Note:**
This is my first pull request ever, if I made a mistake, I'd be glad if you direct me 🙂
(A noter que c’est ma toute première PR sur github, donc si je m’y suis mal pris, hésitez pas à me le faire savoir).
En ce qui concerne la feature, j’ai d’abord pensé à un oublie simple. Mais en regardant le code, tout était déjà prêt. Mais seuls les markers « users » étaient pris en compte. C’est pour ça que j’ai ouvert cette discussion, pensant qu’il y avait une justification à ça.
3 « J'aime »
Merci pour la PR @GziAzman , je viens de tester et ça marche nickel
C’est mergé et ça partira dans la prochaine version de Gladys !
1 « J'aime »
C’est disponible dans Gladys Assistant 4.38.1 :
Salut à tous !
Nouvelle version de Gladys aujourd’hui : Gladys Assistant 4.38.1
Je ferme ce sujet !
1 « J'aime »
)