MTES-MCT / metadata-postgresql

Plume : gestion des métadonnées du patrimoine PostgreSQL
https://mtes-mct.github.io/metadata-postgresql/
GNU Affero General Public License v3.0
1 stars 1 forks source link

ne pas permettre de changer de couche quand on est en mode modification #47

Closed FERRATON closed 2 years ago

FERRATON commented 2 years ago

Si on est en mode modification des métadonnées sur une couche et que l'on change de couche on perd ses modifications sans message d'erreur ou d'alerte

alhyss commented 2 years ago

Il me semble que ça rejoint la question du bouton de verrouillage, qui fait partie des petites améliorations fonctionnelles restant à implémenter - cf. issue #36. Nous l'avions surtout évoqué en ayant en tête le mode lecture, mais il devrait être activé par défaut en mode édition (voire ne pas être désactivable). Tu es d'accord avec ça, @WREATCHED ?

WREATCHED commented 2 years ago

J'essaye de comprendre ton raisonnement @alhyss . Nous avions discuté assez longuement sur le sujet avant que je le code, et l'objectif était de laisser à l'utilisateur la responsabilité de sauvegarder ou pas et de ne pas l'embêter avec une boite de dialogue d'alerte. Maintenant tout est possible, mais il faut prendre le temps de bien le faire Je suis content que la pratique de mettre en option des fonctionnalités soient adoptés et dans ce cas surtout, nous aurons des utilisateurs qui ne veulent pas être pollués par un message et d'autres qui le souhaiteront

WREATCHED commented 2 years ago

Réponse en vidéo

https://user-images.githubusercontent.com/66324136/161423292-1180199e-c6c2-4475-936f-f9aa45f5d3df.mp4

FERRATON commented 2 years ago

image

alhyss commented 2 years ago

Bonjour à tous les deux, Je n'aime pas trop ce fonctionnement. Les messages sont lourds et, si on les désactive, on revient au problème initial. De plus, dans les deux cas l'utilisateur n'est pas libre de manipuler les couches dans son explorateur sans interférence de Plume (l'ouverture des boîtes de dialogue est même encore plus gênante que la situation actuelle de ce point de vue). Il me semblerait préférable de gérer ça dans le cadre du verrouillage - issue #36 -, avec en mode édition un verrouillage activé par défaut et non désactivable qui fera que les actions de l'utilisateur dans le panneau des couches et l'explorateur n'affecteront pas Plume. Dans ce cas il n'y aura évidemment pas lieu d'afficher un quelconque message.

WREATCHED commented 2 years ago