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

Bouton de verrouillage sur une couche ? #36

Closed alhyss closed 2 years ago

alhyss commented 2 years ago

Il me semblerait intéressant d'avoir dans la barre de menu de Plume un bouton de verrouillage qui bloque l'affichage sur la fiche de métadonnées courante quelles que soient les actions de l'utilisateur dans l'explorateur et le panneau des couches.

Cela permettrait à l'utilisateur de se référer à une fiche de métadonnées tout en paramétrant ses couches, plutôt que d'être obligé d'y revenir à chaque fois qu'il manipule une autre couche (Plume basculant alors sur les métadonnées de ladite couche). Je pense que ce serait un vrai plus.

Est-ce que ce serait compliqué à implémenter ?

WREATCHED commented 2 years ago

L'idée est de créer un bouton "magique" de verrouillage et déverrouillage Si verrouillé, alors Plume ne prend plus en compte les interactions avec le gestionnaire de couche et le panneau explorateur Afficher dans la barre de titre de la boite de dialogue principale

WREATCHED commented 2 years ago

Réponse en vidéo https://user-images.githubusercontent.com/66324136/161704992-114fdbc5-a93c-4a46-8f17-c438f10bee0b.mp4

alhyss commented 2 years ago

Ça m'a l'air super !

Comme il faut bien que je trouve à pinailler :

Pour l'icône, tu as fait un SVG ? À la qualité de la vidéo près, elle me semble bien.

WREATCHED commented 2 years ago

Oui, j'ai fait un SVG pour l'icône du cadenas

Résultat

image

alhyss commented 2 years ago

C'est bizarre d'avoir la version seulement quand le verrouillage n'est pas actif. Soit tout le temps, soit jamais me semblerait mieux (puisqu'elle apparaît quand aucune couche PostgreSQL n'est sélectionnée de toute façon). Accessoirement, c'est une amélioration fonctionnelle (et pas la seule en stock) donc elle ira dans Plume 0.4.0, tu peux d'ores-et-déjà mettre ce numéro dans tes métadonnées. Il n'y aura certainement pas de 0.3.3 sauf horrible bug signalé dans les jours qui viennent.

Sinon, je n'aimais pas tellement "verrou sur". Pour nous ça semble clair, mais en fait le verrouillage ne porte pas sur la couche mais sur la fiche de métadonnées affichée par Plume. Mieux vaudrait ne rien écrire qui pourrait suggérer que Plume "bloquerait" la couche elle-même, au risque que l'utilisateur se pose des questions du genre "ai-je le droit de modifier les données de ma couche ?". Si tu ne veux pas de "Couche" comme je le proposais, alors mieux vaut peut-être ne rien mettre : "_Plume | mon_schema.ma_table (mageometrie)". Gagner de la place n'est pas plus mal et l'information sur l'état de verrouillage est déjà portée par le bouton.

D'ailleurs, je n'avais pas fait trop attention aux infobulles, mais toujours dans la même idée.

WREATCHED commented 2 years ago

Tiens il y avait longtemps Euh sinon c'est bien ?

La version, elle est derrière PLUME

image

alhyss commented 2 years ago

Ah, désolée, je n'étais pas claire avec les infobulles. C'était juste le contenu des guillemets que je voulais mettre. Ce n'est pas la peine de citer le nom de la couche dans l'infobulle alors qu'il apparaît déjà dans le titre du panneau.

Et bien sûr que c'est bien !

WREATCHED commented 2 years ago

Pinailleuse c'est féminin ou leslien image

WREATCHED commented 2 years ago

image

alhyss commented 2 years ago

Pour moi, c'est parfait :)