[Issue-294] Ajout du niveau groupe dans l'arborescence #330

LPoin · 2024-11-22T08:25:49Z

Contribution

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

Description

Ajoute le niveau groupe dans l'arborescence des thèmes.

Modifie l'arborescence
Ajoute le drag'n'drop niveau thèmes / groupes / layers
Montée de version de Sortable.js
!!! A merger après [Issue-148] Arborescence du mviewer dans le panneau des thématiques #324 !!!

Elements à modifier / corriger pour validation de cette PR :

Eléments facultatifs :

(Facultatif - Si dans les délais) Après tests et manipulations, il est demandé si on peut rajouter un bouton « Ajouter une donnée » au niveau de chaque groupe pour ajouter directement une donnée dans un groupe sans avoir à la déplacer après l’avoir ajouté. Le bouton « Ajouter une donnée » au niveau de la thématique reste affiché et permet de rajouter une donnée en dehors d’un groupe
(Facultatif) Le comportement natif permet de supprimer une thématique sans modal de confirmation. Si JDev a le temps, il est demandé si on peut rajouter une modal de confirmation pour valider ou non la suppression d’un thème

Éléments à corriger si reproduits :

Contrôle des couches hors groupe et affichage dans le menu des thématiques -> Non reproduit à ce jour

Gaetanbrl · 2024-11-22T08:41:33Z

Ta PR est noté en draft @LPoin, est-ce le cas ?

Gaetanbrl · 2024-11-22T10:42:50Z

@lecault tu peux commencer à tester :)

Gaetanbrl · 2024-11-22T14:11:03Z

!!! A merger après #324 !!!

Rebase à prévoir avec branche develop

lecault · 2024-11-22T16:19:17Z

Premiers retours :

Le nom du groupe n'est pas visible dans la carte :
J'ai réussi à faire un bug d'affichage via le drap and drop (bug qui fait planter le mviewer car groupe vide)

D'ailleurs est-ce un comportement souhaité que mviewer plante si groupe vide ? A première vue, je dirai non.

J'ai eu des soucis d'affichage dans la TOC de couches hors groupes
J'ai eu un souci de drag and drop qui rend impossible tout déplacement

J'ai remarqué d'autres choses que je mettrai en commun avec @smevel

Gaetanbrl · 2024-11-25T07:46:41Z

Le nom du groupe n'est pas visible dans la carte :

En effet, je vois que le group a une propriété title

<group id="group-80405619aa6f" title="Nouveau groupe">

...alors qu'on devrait avoir une propriété name comme ceci :

<group name="Transport maritime" id="grp2" >

Voir cet exemple

J'ai eu des soucis d'affichage dans la TOC de couches hors groupes
J'ai eu un souci de drag and drop qui rend impossible tout déplacement

Nous allons vérifier ce comportement

D'ailleurs est-ce un comportement souhaité que mviewer plante si groupe vide ?

C'est un cas qui sera géré après correction

@LPoin je créer des checkbox dans la description de la PR qui seront à cocher quand ces éléments seront corrigé / vu / validés.

Gaetanbrl · 2024-11-25T07:53:49Z

est-ce un comportement souhaité que mviewer plante si groupe vide ?

Après test, ce comportement provient du mviewer et existait déjà précédemment.
Ce comportement n'est pas souhaitable et est lié à une issue indépendant de cette PR qu'il faut créer et corriger via la communauté.

Gaetanbrl · 2024-11-25T08:10:25Z

Ce comportement n'est pas souhaitable et est lié à une issue indépendant de cette PR qu'il faut créer et corriger

Correction disponible dans la branche develop du mviewer.

Gaetanbrl

1ère phase de revue, des modifications et corrections à apporter :)

lib/mv.js

js/mviewerstudio.js

lib/mv.js

Gaetanbrl · 2024-11-25T11:31:26Z

lib/mv.js

+    getConfGroups: function () {
+      const themeid = mv.getCurrentThemeId();
+      const groupId = `group-${mv.uuid()}`;
+      const groupTitle = "Nouveau groupe";


I18n utilisable ?

Voir ici
https://github.com/mviewer/mviewerstudio/pull/330/files#diff-fbbdf45d15fe208d2883f9619e0098453949e5c3c338cc91532959b79b26554dL498

lib/mv.js

Gaetanbrl · 2024-11-25T11:33:11Z

js/mviewerstudio.js

+                layer["data-themeid"] = toThemeId;
+              }
+            });
+            // Cherche l'index de départ en fonction de si le layer bougé vient d'un groupe ou un thème


Attention aux commentaires en FR (anglais à privilégier)

Gaetanbrl · 2024-11-25T11:54:21Z

Je vois aussi que les infos d'une données ne sont pas toujours chargées dans la modal après avoir fait un drag & drop :

Je n'arrive pas à reproduire systématiquement

lecault · 2024-11-26T14:31:45Z

Je n'ai pas réussi à reproduire le souci d'affichage de couches en dehors de groupes. Je réessayerai une fois les corrections apportées.

…ayer

Gaetanbrl · 2024-12-16T14:30:10Z

plus de souci fonctionnel juste esthétique avec du bleu qui surligne les caractères lorsque l'on déplace une donnée :

Je viens de pousser un commit suite à un teste avec @Agath21. A voir si ca correspond à ce que tu veux @lecault

lecault · 2024-12-16T15:02:47Z

Problème identifié lorsqu'on supprime une donnée localisée dans un groupe :

je met une donnée dans un groupe
je prévisualise
je supprime la donnée
je prévisualise : la donnée est toujours présente

NB : dans le cas où j'ai dans cette thématique une donnée sans groupe, celle-ci est supprimée à la place de l'autre.

lecault · 2024-12-17T09:08:40Z

Si on modifie l'ordre des groupes dans studio, ce n'est pas répercuté dans mviewer.

Côté studio :

Côté mviewer :

Gaetanbrl · 2024-12-17T09:23:39Z

Si on modifie l'ordre des groupes dans studio, ce n'est pas répercuté dans mviewer.

Après analyse, je vois que le groupe qui a été bougé est toujours positionné en dernier index du groupe et peu importe sa position dans l'IHM de l'arborescence :

L'IHM :

Le code lié :

Ce qui est attendu :

Repositionné le groupe à l'index ciblé par le drag & drop (au drop donc)

LPoin · 2024-12-17T09:40:59Z

Je viens de regarder, dans mviewerstudio.js ligne 524 dans le onEnd des groupes, ça push dans config.themes[toThemeId].groups mais j'ai oublié d'ajouter la notion d'index comme pour les layers (config.themes[toThemeId].layers.splice(newIndex, 0, itemToMove) )
Ca push donc toujours à la fin.

Gaetanbrl · 2024-12-17T09:41:59Z

Je viens de regarder, dans mviewerstudio.js ligne 524 dans le onEnd des groupes, ça push dans config.themes[toThemeId].groups mais j'ai oublié d'ajouter la notion d'index comme pour les layers (config.themes[toThemeId].layers.splice(newIndex, 0, itemToMove) ) Ca push donc toujours à la fin.

Oui c'est corrigé :)

lecault

Ok pour moi

Gaetanbrl · 2024-12-17T10:15:48Z

A merger en mode Squash and merge @LPoin 👍

LPoin · 2024-12-17T10:19:32Z

Je n'ai pas les droits pour merger moi.

Gaetanbrl · 2024-12-17T10:21:10Z

La PR est validée, merci à tous pour les développements, suivis et tests.

LPoin added 4 commits November 15, 2024 10:06

modify the drag n drop and groups layer in themes

124ccce

wip enregistrement groupes

a86d7e0

add save and load theme, groups and layers

50bf662

clean code

a9cd59f

LPoin marked this pull request as draft November 22, 2024 08:26

LPoin mentioned this pull request Nov 22, 2024

Ajout du niveau groupe dans l'arborescence #294

Closed

Gaetanbrl requested review from lecault and Gaetanbrl November 22, 2024 10:42

clean code and graphic design

9e20c59

LPoin marked this pull request as ready for review November 22, 2024 11:47

lecault added the enhancement label Nov 22, 2024

lecault added this to the 4.2 milestone Nov 22, 2024

Gaetanbrl requested changes Nov 25, 2024

View reviewed changes

LPoin added 8 commits November 26, 2024 17:04

modifie le title en name dans les groupes

2bffc43

display groups panel depending on whether or not a group is present

09e01ad

Modify button title for add theme, groups and layers

b8e6d5a

issue 294 - clean delete layer in a group

3dca0ca

issue 294 - layer edition in a group && save studio at save editing l…

2ce63c1

…ayer

modify the drag n drop and groups layer in themes

3321edb

wip enregistrement groupes

058e979

add save and load theme, groups and layers

5d2ec0a

Gaetanbrl added 5 commits December 13, 2024 14:35

fix mviewer#344 - get layer by ID from group and theme

3e35803

pretty

0268d12

mviewer#341 - pass layer as str XML correctly

3bbd945

pretty

8e0e1a4

fix error with attributefilter

d1301ad

lecault mentioned this pull request Dec 13, 2024

Migration 4.1 => 4.2 : ouverture d'applications existantes #342

Closed

Gaetanbrl mentioned this pull request Dec 16, 2024

Import d'un thème XML : thématiques non visibles #341

Closed

lecault mentioned this pull request Dec 16, 2024

L'option réduire le panneau des thématiques ne fonctionne pas correctement #339

Closed

Fix groupId error and showintoc default state mviewer#323 mviewer#330

a0520af

Gaetanbrl self-requested a review December 16, 2024 09:42

hide blue user select from draggable tree

a10e9f6

Gaetanbrl added 2 commits December 16, 2024 15:43

ease scroll on drag event mviewer#345

fa46bbe

fix overflow y value mviewer#345

e01c2c9

Gaetanbrl added 2 commits December 16, 2024 16:17

fix delete layer from group

cf40969

remove height css constraint

5b83411

Fix group index on drag drop

3b9e772

lecault approved these changes Dec 17, 2024

View reviewed changes

Gaetanbrl approved these changes Dec 17, 2024

View reviewed changes

Gaetanbrl merged commit 83d3153 into mviewer:develop Dec 17, 2024
1 check passed

Gaetanbrl mentioned this pull request Dec 17, 2024

Légende disparue suite à publication d'une application #323

Closed

lecault mentioned this pull request Jan 9, 2025

Version 4.2 #361

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Issue-294] Ajout du niveau groupe dans l'arborescence #330

[Issue-294] Ajout du niveau groupe dans l'arborescence #330

LPoin commented Nov 22, 2024 •

edited by Gaetanbrl

Loading

Gaetanbrl commented Nov 22, 2024

Gaetanbrl commented Nov 22, 2024

Gaetanbrl commented Nov 22, 2024

lecault commented Nov 22, 2024

Gaetanbrl commented Nov 25, 2024

Gaetanbrl commented Nov 25, 2024

Gaetanbrl commented Nov 25, 2024

Gaetanbrl left a comment

Gaetanbrl Nov 25, 2024

Gaetanbrl Nov 25, 2024

Gaetanbrl Nov 25, 2024

Gaetanbrl commented Nov 25, 2024 •

edited

Loading

lecault commented Nov 26, 2024

Gaetanbrl commented Dec 16, 2024

lecault commented Dec 16, 2024 •

edited

Loading

lecault commented Dec 17, 2024

Gaetanbrl commented Dec 17, 2024 •

edited

Loading

LPoin commented Dec 17, 2024 •

edited

Loading

Gaetanbrl commented Dec 17, 2024

lecault left a comment

Gaetanbrl commented Dec 17, 2024

LPoin commented Dec 17, 2024

Gaetanbrl commented Dec 17, 2024

[Issue-294] Ajout du niveau groupe dans l'arborescence #330

[Issue-294] Ajout du niveau groupe dans l'arborescence #330

Conversation

LPoin commented Nov 22, 2024 • edited by Gaetanbrl Loading

Contribution

Description

Gaetanbrl commented Nov 22, 2024

Gaetanbrl commented Nov 22, 2024

Gaetanbrl commented Nov 22, 2024

lecault commented Nov 22, 2024

Gaetanbrl commented Nov 25, 2024

Gaetanbrl commented Nov 25, 2024

Gaetanbrl commented Nov 25, 2024

Gaetanbrl left a comment

Choose a reason for hiding this comment

Gaetanbrl Nov 25, 2024

Choose a reason for hiding this comment

Gaetanbrl Nov 25, 2024

Choose a reason for hiding this comment

Gaetanbrl Nov 25, 2024

Choose a reason for hiding this comment

Gaetanbrl commented Nov 25, 2024 • edited Loading

lecault commented Nov 26, 2024

Gaetanbrl commented Dec 16, 2024

lecault commented Dec 16, 2024 • edited Loading

lecault commented Dec 17, 2024

Gaetanbrl commented Dec 17, 2024 • edited Loading

LPoin commented Dec 17, 2024 • edited Loading

Gaetanbrl commented Dec 17, 2024

lecault left a comment

Choose a reason for hiding this comment

Gaetanbrl commented Dec 17, 2024

LPoin commented Dec 17, 2024

Gaetanbrl commented Dec 17, 2024

LPoin commented Nov 22, 2024 •

edited by Gaetanbrl

Loading

Gaetanbrl commented Nov 25, 2024 •

edited

Loading

lecault commented Dec 16, 2024 •

edited

Loading

Gaetanbrl commented Dec 17, 2024 •

edited

Loading

LPoin commented Dec 17, 2024 •

edited

Loading