Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MEG - Cohérence mviewerstudio/mviewer #157

Closed
21 tasks done
Gaetanbrl opened this issue Feb 3, 2023 · 48 comments
Closed
21 tasks done

MEG - Cohérence mviewerstudio/mviewer #157

Gaetanbrl opened this issue Feb 3, 2023 · 48 comments
Assignees
Milestone

Comments

@Gaetanbrl
Copy link
Member

Gaetanbrl commented Feb 3, 2023

Issue développée dans le cadre d'une prestation JDev / Megalis Bretagne / GeoBretagne

Histoire

En tant que utilisateur,
Je souhaite avoir les nouveaux paramètres mviewer dans l'interface mviewerstudio
Pour ne pas avoir à modifier la configuration XML et n'utiliser que l'interface mviewerstudio.

Description détaillée

Paramètres de l'application

PARAMETRE COMPOSANT CIBLE Type Mode avancé
Zoomtools Checkbox Onglet Application bool
Initialextenttool Checkbox Onglet Application bool
Titlehelp Champ text Onglet Application string X
Iconhelp Bouton + modale Onglet Application list X
Geoloc Checkbox Onglet Application bool
Mouseposition Checkbox Onglet Application bool X
Studio Champ text Onglet Application string X
Home Champ text Onglet Application string X
Favicon Champ text Onglet Application string X

Paramètres de couche

PARAMETRE COMPOSANT Cible Type Mode avancé
Index Champ numérique Couche / Onglet avancées / options d'affichage integer X
Styletitle Champ text Couche / Onglet général string
dynamiclegend Checkbox Couche / Onglet avancées / options d'affichage bool X
exclusive Checkbox Couche / Onglet avancées / options d'affichage bool X
toplayer Checkbox Couche / Onglet général / options d'affichage bool
showintoc Checkbox Couche / Onglet avancées / options d'affichage bool X

Avancement

https://github.com/jdev-org/mviewerstudio/tree/issue-157

  • Paramètres de l'application
  • Paramètres de couche
  • Tous
  • Documentation associées aux paramètres - @Gaetanbrl @Agath21
  • Mise à jour de la documentation mviewer avec le badge "studio" - @Gaetanbrl @Agath21
  • Revue des placeholder (les rendre + explicite)

Bonus

@spelhate
Copy link
Collaborator

spelhate commented Feb 6, 2023

On pourrait également enregistrer la version utilisée du studio dans le XML généré et la version cible du mviewer ?

@Gaetanbrl
Copy link
Member Author

On pourrait également enregistrer la version utilisée du studio dans le XML généré et la version cible du mviewer ?

Est-ce un paramètre supplémentaire de cette issue @spelhate ?

@Gaetanbrl
Copy link
Member Author

@spelhate @lecault pouvez-vous compléter la colonne Mode avancé qui permet de définir quels sont les paramètres visibles en mode avancés et lesquels sont visibles en mode simple ?

@Gaetanbrl
Copy link
Member Author

pouvez-vous compléter la colonne Mode avancé

Complété en COTECH le 06/02/2023

@Agath21
Copy link
Collaborator

Agath21 commented Feb 6, 2023

Proposition technique - Ajout des fonctionnalités au sein de l'interface

Module Application

Studio2_applicationAddOptions

Module Thématiques & données / Paramètres de la donnée / Onglet "Général"

Studio2_dataGeneral

Module Thématiques & données / Paramètres de la donnée / Onglet "Options avancées"

Studio2_dataOptionsAvancees

Gestion de la légende :

  • Ajout d'un nouvel accordéon pour la gestion de la légende avec la fonctionnalité "Légende statique" (fonctionnalité existante déplacée de "Options d'affichage") + switch de légende dynamique + Légende personnalisée (img)
  • Ajout d'une liste déroulante pour le type de légende
  • Par défaut, c'est la légende geoserver issue du SLD
  • Avec la valeur "personnalisée", le switch de la légende dynamique est remplacé par un input permettant d'indiquer un lien vers une image :
    Studio2_legende_personnalisée

@lecault
Copy link
Collaborator

lecault commented Feb 7, 2023

Concernant la légende, peux on voit l'interface en mode normal ?

@Agath21
Copy link
Collaborator

Agath21 commented Feb 7, 2023

@lecault Les paramètres liés à la légende étant situés dans l'onglet "Options avancées", ils ne sont pas accessibles / visibles en mode simple.
image

@lecault
Copy link
Collaborator

lecault commented Feb 7, 2023

Ah d'accord. Je voulais voir l'affichage si type de légende <> personnalisée

@Agath21
Copy link
Collaborator

Agath21 commented Feb 7, 2023

Désole, je n'avais pas compris ta demande.

De base, c'est la légende geoserver avec le type de légende est "Par défaut". Seul l'option "Adapter la légende selon les données sur la carte" est disponible. Cette option correspond actuellement à la fonctionnalité "Légende dynamique" mais nous avons modifié l'intitulé que nous trouvions peu parlant. Le visuel est disponible dans mon commentaire avec les maquettes ou en zoom ci-dessous :

image

@lecault
Copy link
Collaborator

lecault commented Feb 7, 2023

Je reformulerai en "Adapter la légende selon le zoom de la carte"

@Agath21
Copy link
Collaborator

Agath21 commented Feb 7, 2023

C'est noté, on se basera sur cet intitulé.

@lecault
Copy link
Collaborator

lecault commented Feb 7, 2023

A moins que je n'ai pas anticipé d'autres usages @spelhate mais pour moi ça ne dépend que du zoom.

Agath21 added a commit to jdev-org/mviewerstudio that referenced this issue Feb 7, 2023
@Agath21
Copy link
Collaborator

Agath21 commented Feb 7, 2023

Identification d'un bug Mviewer | Options de l'application

Description

Lorsque l'on ajoute les paramètres "geoloc" et "studio" dans la balise application du config.xml, il s'affiche constamment dans l'application malgré le changement de valeurs entre false/true . Pour les désactiver, il faut les supprimer totalement du config.xml

image

image

Exemple en ligne :
App > https://gis.jdev.fr/mviewer/?config=apps/a00f85e7953de66500f54760a9522867.xml
config.xml > https://gis.jdev.fr/mviewer/apps/a00f85e7953de66500f54760a9522867.xml

Incidence sur le développement en cours

Les paramètres "geoloc" et "studio" sont déclarés et donc intégrés dans le config.xml généré par le studio. Les valeurs false/true changent en fonction des checkbox activées (ok > vérification du config généré et téléchargé).

Toutefois, avec ce bug les deux fonctionnalités sont toujours activées dans les applications mviewer.

Agath21 added a commit to jdev-org/mviewerstudio that referenced this issue Feb 7, 2023
@Gaetanbrl
Copy link
Member Author

Identification d'un bug Mviewer

@Agath21 :

  • bug à reporter dans le bon dépôt
  • lien vers cette issue (ou commentaire précédent) à rajouter dans l'issue du bug

Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue Feb 9, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue Feb 9, 2023
@spelhate
Copy link
Collaborator

spelhate commented Feb 9, 2023

On pourrait également enregistrer la version utilisée du studio dans le XML généré et la version cible du mviewer ?

Est-ce un paramètre supplémentaire de cette issue @spelhate ?

Il ne s'agit pas d'un paramètre, mais d'un attribut associé au config.xml exemple :

<?xml version="1.0" encoding="UTF-8"?>
<config versionstudio="3.2">

@spelhate
Copy link
Collaborator

spelhate commented Feb 9, 2023

A moins que je n'ai pas anticipé d'autres usages @spelhate mais pour moi ça ne dépend que du zoom.

Oui c'est bien ça.

@spelhate
Copy link
Collaborator

spelhate commented Feb 9, 2023

Il faudra également penser à mettre à jour la doc mviewer avec l'indication studio en face des paramètres concernés :

image
https://mviewerdoc.readthedocs.io/fr/latest/doc_tech/config_app.html#parametres-principaux

@spelhate
Copy link
Collaborator

spelhate commented Feb 9, 2023

Identification d'un bug Mviewer

@Agath21 :

* bug à reporter dans le bon dépôt

* lien vers cette issue (ou commentaire précédent) à rajouter dans l'issue du bug

J'ai créé l'issue et la PR qui corrige le bug (mviewer/mviewer#733)

@Agath21
Copy link
Collaborator

Agath21 commented Mar 20, 2023

Remarque sur le bouton éditer dans studio. Il renvoie vers la page d'accueil du studio via une URL de ce type : http://test-sig5/mviewerstudio/index.html#http://test-sig5/mviewer/apps/store/39a9d224040f/preview/c505368e-c412-11ed-8811-00505698285b.xml

Il serait souhaitable d'arriver sur le studio avec les paramètres de l'appli chargée.

Problème spécifié dans l'issue #185

@lecault
Copy link
Collaborator

lecault commented Mar 20, 2023

Suite à démo à mes collègues, une reformulation à faire :
Affichage de la donnée seulement sur la carte => Ne pas afficher la donnée dans la légende

@Agath21
Copy link
Collaborator

Agath21 commented Mar 22, 2023

Suite à démo à mes collègues, une reformulation à faire :
Affichage de la donnée seulement sur la carte => Ne pas afficher la donnée dans la légende

Ok, modifié ce jour

@Agath21 Agath21 closed this as completed Mar 23, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
@Gaetanbrl Gaetanbrl added this to the MEGALIS - JDEV milestone Jun 16, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Gaetanbrl pushed a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants