Skip to content

Update the way to access map management#653

Merged
AerunDev merged 12 commits intodevelopfrom
feat-646-update-way-maps-management
Oct 11, 2025
Merged

Update the way to access map management#653
AerunDev merged 12 commits intodevelopfrom
feat-646-update-way-maps-management

Conversation

@AntoinePoree
Copy link
Collaborator

@AntoinePoree AntoinePoree commented Sep 4, 2025

Description

Review maps router to access maps mangagement and Event

  • Update maps router
  • Add translation EN - FR

Close #646

Copy link
Collaborator

@AerunDev AerunDev left a comment

Choose a reason for hiding this comment

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

Hello ! 😄

J'ai pu tester ta MR, et c'est vraiment top, bien joué.
J'ai corrigé deux choses directement en faisant un commit :

  • Le wording sur le placeholder de la barre de recherche des événements, qui affichait "Recherche" alors que celle des cartes affichait "Rechercher une carte...". C'est maintenant "Rechercher un événement..." avec une clé dédiée.
  • La page des maplinks pointait vers /world/maplinks alors que le chemin est /world/maplink sans s.

Changements à apporter

  • Le sous-menu "Liaison des cartes" ne doit être accessible que quand on est sur la Tab "Cartes". L'idée étant d'avoir un contexte et un sous-menu adapté. Par exemple : "Templates des cartes" ou "Templates des événements".
  • Quand on est sur la Tab "Événements", la liste est vide pour le moment et je pense que ça va perturber les utilisateurs. Il faudrait afficher dans la liste quelque chose comme "Aucun événement.".
    • À voir d'ailleurs si on généralise à la liste des cartes pour "Aucune carte.".

Communication en cas de liste des événements vide

En faisant mes tests, je me suis dit qu'il serait super intéressant de profiter de l'état vide de la liste des cartes (voir le design sur Figma, le sous-menu de gauche est bien là, il manque juste sur la maquette) pour profiter de ce composant et informer l'utilisateur que c'est normal que sa liste des événements soit vide pour le moment.

Pour ça, je propose ce type de Design (voir le design sur Figma, et voir la version anglaise sur Figma), à discuter ensemble.

Si on veut éviter des questions et incompréhensions des utilisateurs, il vaut mieux ajouter cette partie qui permettra d'être clair sur nos projets pour la V3, dès la consultation de l'interface.
On fera évoluer cette partie une fois la V3 sortie ! 😄

Problématique de CSS

Voir ensemble si c'est à faire maintenant ou dans un ticket dédié.

J'ai remarqué aussi un comportement dû à la façon dont le CSS est géré sur les noms des cartes et dossiers du composant de la liste des cartes, et je te mets une capture d'écran ici pour mieux visualiser.

  • Quand je me rends sur la liste des cartes,
  • Et que je mets mon curseur sur un bouton d'une carte,
  • Alors le nom est trop grand il est tronqué et une ellipsis est affichée,
  • Sauf que l'espace entre l'ellipsis et les boutons d'action à droite est anormalement grand.

C'est peut-être pas dans le scope de cette PR, mais je me demande si c'est lié à l'agrandissement de la largeur du composant.
Est-ce qu'on ne pourrait pas faire sauter tous ces calculs de max-width en dur dans MapTreeStyle.tsx pour utiliser un fill-container et faciliter le fait que si un jour on veut pouvoir resize comme on le souhaite ce soit responsive ?

@AntoinePoree
Copy link
Collaborator Author

Hello ! 😄

J'ai pu tester ta MR, et c'est vraiment top, bien joué. J'ai corrigé deux choses directement en faisant un commit :

  • Le wording sur le placeholder de la barre de recherche des événements, qui affichait "Recherche" alors que celle des cartes affichait "Rechercher une carte...". C'est maintenant "Rechercher un événement..." avec une clé dédiée.
  • La page des maplinks pointait vers /world/maplinks alors que le chemin est /world/maplink sans s.

Changements à apporter

  • Le sous-menu "Liaison des cartes" ne doit être accessible que quand on est sur la Tab "Cartes". L'idée étant d'avoir un contexte et un sous-menu adapté. Par exemple : "Templates des cartes" ou "Templates des événements".

  • Quand on est sur la Tab "Événements", la liste est vide pour le moment et je pense que ça va perturber les utilisateurs. Il faudrait afficher dans la liste quelque chose comme "Aucun événement.".

    • À voir d'ailleurs si on généralise à la liste des cartes pour "Aucune carte.".

Communication en cas de liste des événements vide

En faisant mes tests, je me suis dit qu'il serait super intéressant de profiter de l'état vide de la liste des cartes (voir le design sur Figma, le sous-menu de gauche est bien là, il manque juste sur la maquette) pour profiter de ce composant et informer l'utilisateur que c'est normal que sa liste des événements soit vide pour le moment.

Pour ça, je propose ce type de Design (voir le design sur Figma, et voir la version anglaise sur Figma), à discuter ensemble.

Si on veut éviter des questions et incompréhensions des utilisateurs, il vaut mieux ajouter cette partie qui permettra d'être clair sur nos projets pour la V3, dès la consultation de l'interface. On fera évoluer cette partie une fois la V3 sortie ! 😄

Problématique de CSS

Voir ensemble si c'est à faire maintenant ou dans un ticket dédié.

J'ai remarqué aussi un comportement dû à la façon dont le CSS est géré sur les noms des cartes et dossiers du composant de la liste des cartes, et je te mets une capture d'écran ici pour mieux visualiser.

  • Quand je me rends sur la liste des cartes,
  • Et que je mets mon curseur sur un bouton d'une carte,
  • Alors le nom est trop grand il est tronqué et une ellipsis est affichée,
  • Sauf que l'espace entre l'ellipsis et les boutons d'action à droite est anormalement grand.

C'est peut-être pas dans le scope de cette PR, mais je me demande si c'est lié à l'agrandissement de la largeur du composant. Est-ce qu'on ne pourrait pas faire sauter tous ces calculs de max-width en dur dans MapTreeStyle.tsx pour utiliser un fill-container et faciliter le fait que si un jour on veut pouvoir resize comme on le souhaite ce soit responsive ?

Premièrement, merci d'avoir corrigé mes erreurs de labels et wording Aerun. 😇

Pour la suite, j'ai ajouté un label "Aucun évenement" et "Aucune map" sur les deux composants d'event et de Mapthree, avec un commentaire pour l'intégrer proprement dans le futur Event three. L'harmonisation est importante donc j'ai pris la liberté de le faire. A voir si cela te convient et les espaces sont prévus pour un composant custom .

J'ai revu le css pour que cela soit meilleure responsivement, cela fonctionne correctement sur mon poste, donc je vous laisse la liberté de le testé avec des dossiers de sous map avec 4 itérations 😢

Copy link
Collaborator

@AerunDev AerunDev left a comment

Choose a reason for hiding this comment

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

J'ai pu tester avec tes dernières modifications @AntoinePoree et ça me paraît nickel comme ça !

J'ai juste remarqué un souci de CSS quand on tape quelque chose dans l'input de recherche des événements : le texte "Aucun événement" semble bouger sans raison.
Je pense aussi que le fait que le texte soit une simple balise "p" sans style sera à corriger pour utiliser le style habituel des textes. Il faudrait d'ailleurs voir si on a pas déjà des "Aucun xxx" quelque part, parce que je crois qu'on les affiche plutôt en text400 en général.

Je laisserai un dev approuver et si tout est OK on pourra merge dès que possible.
A voir si on veut corriger ces détails de style avant pour que ce soit nickel.

Pour info, en attendant qu'on ait le composant d'état Empty dont je parle plus haut, on va sortir une release sans cette PR, et la merge direct après comme ça les utilisateurs ne seront pas perdus. Là en l'état avec les boutons non fonctionnels on va se faire mitrailler de questions du genre "pourquoi ça marche pas ?".

@AerunDev AerunDev changed the title feat: update the way to access map management Update the way to access map management Sep 29, 2025
Copy link
Collaborator

@AerunDev AerunDev left a comment

Choose a reason for hiding this comment

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

Je viens de tester dans le doute après ton dernier commit et j'ai trouvé un petit bug embêtant pour les utilisateurs qui s'amuseraient à cliquer plusieurs fois sur le bouton "Nouveau dossier" du EventTree. En effet, comme c'est toujours branché à la partie MapInfo, ça créé des dossiers côté MapTree.

Pour simplifier les choses, j'ai fait en sorte que les boutons soient désactivés si on est pas en mode dev 😄

@Otaku17 va bosser sur le composant d'info en cas de liste des événements vide (#656), et on pourra release sans problème vos deux MR ! 🎉

@Palbolsky je te laisse review les derniers changements et on peut merge !

@AerunDev
Copy link
Collaborator

AerunDev commented Oct 7, 2025

Si quelqu'un est motivé pour faire une revue de code des derniers changements, on pourra merge cette PR ! 😄

@AerunDev AerunDev merged commit d41292f into develop Oct 11, 2025
5 checks passed
@AerunDev AerunDev deleted the feat-646-update-way-maps-management branch October 11, 2025 19:00
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.

Update the way to access maps management

4 participants