[Issue-148] Arborescence du mviewer dans le panneau des thématiques

[LPoin]

Contribution

Cette contribution est financée par la Région Bretagne :

Description

Modifie l'arborescence des Thèmatiques & Données pour supprimer la modale de création et modification des thèmes et couches.

• Gestion par ID et non plus par élément actif
• Refacto du code concerné

[lecault]

Merci, les tests sont réalisés ici : https://kartenn.region-bretagne.fr/mviewerstudio-dev/index.html#

• Je n'arrive pas à récupérer les pictos des thématiques :
  

• L'option déroulé par défaut n'est pas pris en compte si on ne clique pas sur enregistrer

• La modification du nom de la thématique n'est pas pris en compte si on ne clique pas sur enregistrer

• Le retour à la ligne est autorisé dans le nommage de la thématique ce qui ne devrait pas être le cas
  

• Ok pour le rafraîchissement du nom des couches

• Comme dit dans l'ancienne PR, je n'ai pas tenu compte des drag and drop

[Gaetanbrl]

Je n'arrive pas à récupérer les pictos des thématiques :

Ce problème devrait maintenant être corrigé dans cette branche.

[Gaetanbrl]

L'option déroulé par défaut n'est pas pris en compte si on ne clique pas sur enregistrer
La modification du nom de la thématique n'est pas pris en compte si on ne clique pas sur enregistrer

Ce sont des remarques pour modification ou des questions ?

En l'état je trouve ça normal de ne pas enregistrer à chaque modification. L'utilisateur devrait faire ses modifications et sauvegarder quand il le souhaite à la fin ou bien annuler ses modifications.
Si on sauvegarde à chaque modification, il faut que l'utilisateur identifie la version avant ses modifications (ca en fera une liste...) et la sélectionne comme version courante.

Je ne suis pas certain que ca facilite ce cas d'usage.

[Gaetanbrl]

Après un petit nettoyage à faire (code commenté / console.log) ce sera bon pour le coté code.

[lecault]

L'option déroulé par défaut n'est pas pris en compte si on ne clique pas sur enregistrer
La modification du nom de la thématique n'est pas pris en compte si on ne clique pas sur enregistrer

Ce sont des remarques pour modification ou des questions ?

En l'état je trouve ça normal de ne pas enregistrer à chaque modification. L'utilisateur devrait faire ses modifications et sauvegarder quand il le souhaite à la fin ou bien annuler ses modifications. Si on sauvegarde à chaque modification, il faut que l'utilisateur identifie la version avant ses modifications (ca en fera une liste...) et la sélectionne comme version courante.

Je ne suis pas certain que ca facilite ce cas d'usage.

On a discuté avec Stéphane.
Sur la version précédente, la modal thématique enregistrait automatiquement les modifications sans cliquer sur enregistrer, c'est pour cela qu'on est perturbé par ce nouveau fonctionnement.

Notre idée : le clic sur preview enregistre le projet s'il y a des modifications et ne l'enregistre pas s'il n'y a pas de modifications.
Qu'en pensez vous ?

[lecault]

Choisir ce picto par défaut pour les thématiques :

[Gaetanbrl]

Notre idée : le clic sur preview enregistre le projet s'il y a des modifications et ne l'enregistre pas s'il n'y a pas de modifications.
Qu'en pensez vous ?

Soit on reste sur le fonctionnement initial soit on fait une nouvelle évolution.

[Gaetanbrl]

Choisir ce picto par défaut pour les thématiques :

Tu es certain que c'est le plus adapté ?

[lecault]

Choisir ce picto par défaut pour les thématiques :

Tu es certain que c'est le plus adapté ?

Oui c'est celui qu'on avait historiquement

[LPoin]

Push des modifs selon les retours de PR.

[lecault]

Merci pour les modifications :

• OK pour la sauvegarde via clic sur preview
• Zone de modification du titre plus lisible
• Reste à modifier le picto par défaut

[Gaetanbrl]

Reste à modifier le picto par défaut

Ok on merge une fois cette modification faite.
Merci @LPoin @lecault

[Gaetanbrl]

@lecault je te laisse faire le squash and merge si c'est bon pour toi