Skip to content

feat: add map init options + zoom and center getters#33

Merged
arnaudfnr merged 11 commits intomainfrom
feat/init-map-options
Feb 27, 2026
Merged

feat: add map init options + zoom and center getters#33
arnaudfnr merged 11 commits intomainfrom
feat/init-map-options

Conversation

@jbuiquan
Copy link
Copy Markdown
Collaborator

@jbuiquan jbuiquan commented Feb 25, 2026

Issue #26

Remarques:

@jbuiquan jbuiquan marked this pull request as ready for review February 26, 2026 08:37
@jbuiquan jbuiquan self-assigned this Feb 26, 2026
arnaudfnr and others added 6 commits February 26, 2026 21:02
* Chore( Add biome lint config and fix errors )

* Remove unnecessary changes

* Fix 1 more warning

* Add github worklow for Code Quality check

* remove yarn cache

* Add lint error to check github worklflow

* Remove error and trigger workflow only once

* Change run command to also check format

* Fix review
Copy link
Copy Markdown
Collaborator

@Cabanon Cabanon left a comment

Choose a reason for hiding this comment

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

C'est pour quoi faire d'avoir le centre et le zoom ? Est-ce que vous avez besoin d'avoir aussi la possibilité de zoomer pour avoir tous les points visibles ?

Comment thread coordo-ts/src/index.ts Outdated
url?: string;
};
type MapOptions = {
zoom?: number;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Pas besoin de redéfinir ce type, il doit être dans maplibre

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@arnaudfnr
Copy link
Copy Markdown
Collaborator

C'est pour quoi faire d'avoir le centre et le zoom ? Est-ce que vous avez besoin d'avoir aussi la possibilité de zoomer pour avoir tous les points visibles ?

C'est pour que la carte sur le site web apparaisse directement avec un zoom et un centre que l'on peut définir au moment de créer la Map, par exemple sur Madagascar directement. Côté d4t on va stocker le zoom et le centre pour que quand un utilisateur bouge la carte, ferme l'onglet, puis retourne sur le site, il revienne à l'emplacement ou il s'était arrêté lorsqu'il avait fermé l'onglet. C'est un petit feat sympathique a avoir

@arnaudfnr arnaudfnr merged commit b4c7168 into main Feb 27, 2026
2 checks passed
@arnaudfnr arnaudfnr deleted the feat/init-map-options branch March 18, 2026 12:19
Cabanon pushed a commit that referenced this pull request Mar 19, 2026
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.

[Coordo Frontend] Allow the map to be initialized with customized zoom and center

4 participants