✨ Revue UI complète des widgets/boutons/panels

[iamvdo]

Revue de code UI complète de l'intégration des widgets pour respect des maquettes et corrections de bugs.

Pour la revue UI, dans les grandes lignes:

• architecture globale en CSS Grid (juste 3 ou 4 lignes possibles en fait)
• passage des tailles de bouton à 48x48
• largeur et hauteur des panneaux min/max (évite les débordements)
• les panneaux s'ouvrent en haut, du coté du widget
• gestion du nombre de widgets "outils" en fonction de la hauteur
• utilisation de variables Sass (tailles, styles, media-queries)
• règles de base dans carte/controls/Controls.vue, règles spécifiques dans le composant de chaque widget

Aussi, j'ai mis à jour le panel MenuControl en UI/UX. Pour l'UX, j'ai supprimé les DsfrTooltip. Elias m'a parlé du ticket #429, mais je préfère cette solution. Reste notamment le cas de description facultative (comme mentionné dans le ticket), qui à mon avis est une bonne idée.

Voilà une liste de ticket que ça corrige (à recetter ?):

• UI(panels): rationnaliser largeur panels #672 (mais pas le contenu du panel "Mes enregistrements")
• Zoom : améliorations Ui #700 (?)
• UI (boutons et icônes) #743 (?)
• Widget mini carte #794
• [bug] Gestionnaire de couches/layerSwitcher : sur edge la modale passe derrière le footer #844
• Bug(MenuWidget): les tooltips sont persistantes si Eulerian activé #866 (en virant les tooltips)
• [bug] sélecteur de territoires (territories) : modale dépasse de l'écran #886
• l'affichage des couches dans le gestionnaires de couches est masqué par le footer #888

Autres informations

J'ai pas encore mis à jour le changelog.

[lowzonenose]

pas de review car ça pique !
mais, ça me semble beaucoup plus claire 🥇

[iamvdo]

Un vrai point à discuter: pour l'instant, j'ai tout mis ici, dans l'entrée carto mais il serait préférable d'avoir certains styles directement coté extensions (notamment les tailles min/max des panels)

[elias75015]

Mathis avait commencé une reflexion sur ce sujet côté ext:
IGNF/geopf-extensions-openlayers#467

[elias75015]

Beau boulot !!

Petits retours (ce que j'ai pour l'instant) :

• Le widget "..." n'est censé apparaitre que s'il n'y a pas assez de place en hauteur pour afficher tous les boutons.
  Or là il apparait même quand toute la place n'est pas occupée par les boutons verticalement :

• Les boutons de zoom sont censés être désactivés par défaut en mode mobile (voir ce ticket). Si je simule mode mobile + localStorage.clear() et rechargement de la page, les boutons de zoom aparaissent. Bon, en prod c'est aussi buggé, donc c'est pas si pire avec ta PR : les boutons de zoom n'apparaissent jamais en mode mobile, mais l'entrée reste disponible dans le menu des widgets...

[elias75015]

La modale de partage est coupée, et donc le partage par iframe inutilisable :

Pareil pour la modale d'impression :

[elias75015]

Voir commentaires ci-dessus : principalement problème sur les modales print + partage avec hauteur bloquée

[elias75015]

Good pour moi, merci !
Il faudrait peut être mettre à jour les extensions incorporées à cette branche pour corriger le bouton territories

[iamvdo]

J'ai finalisé le menu des controles: j'ai repris les textes de la #429 + maquettes, organisé par groupes (c'est dans le dernier commit)
Pour les extensions, tu veux faire une release avant ?

[elias75015]

release 1.0.0-beta.9 des extensions en cours de publication. On pourra monter ctte version sur cette branche

[elias75015]

Pour référencement : tri des outils en catégories issu de la maquette