-
Notifications
You must be signed in to change notification settings - Fork 32
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
Dans l'outil de sélection multiple ce serait pratique d'avoir l'oeil, pour activer ou désactiver plusieurs couches en même temps #453
Comments
L'amélioration 453 ajoute une nouvelle fonctionnalité. = = Dans igo2 = = Sous le menu "Carte", on trouve les options pour les "Couches" et les "Légendes".
Si on coche sur la case à cocher [au point 4], une barre d'outils pour les options de la sélection multiple apparaît après la fin de la liste. = = Dans igo2-lib = = Sous le titre de partie "Layers", on trouve une liste de layer ou couches.
Si on coche sur la case à cocher [4], une barre d'outils pour les options de la sélection multiple apparaît dans le pied de page. igo2-lib fournit un lien vers un exemple de code source pour invoquer la fonctionnalité. Le code source du composant "igo-layer-list" se trouve dans le répertoire suivant : igo2-lib/packages/geo/src/lib/layer/layer-list/
Lorsque c'est modifié, on peut se contenter de recompiler le module geo : Le changement sera visible dans igo2-lib, mais pas encore dans igo2 Il faut interrompre igo2 (localhost:4201) |
Analyse des spécificationsOn demande un oeil pour pour activer ou désactiver plusieurs couches en même temps dans la sélection multiple. Si on ajoute un oeil qui permet de cacher les couches sélectionnés, une question se pose. Que faire avec les couches déjà cachées individuellement ? Je propose que l'oeil permette d'inverser la visibilité, pour contourner la difficulté. Ce comportement me semble plus simple à comprendre. Il permet d'évaluer visuellement des alternatives possibles. Exemple A |
Après avoir consulté Marc-André Barbeau, la solution simple discutée plus haut a été retenue. Exemple A On clique sur l'oeil On re-clique sur l'oeil |
Vote sur le comportement de l'oeil avec cette situation: Solution A (Voter 🚀 ): Solution B (Voter ❤️ ): |
Lorsque toute les couches sont non visibles afficher un oeil barré pour icone. Exemple : icone oeil noir dans la bar utilisateur clique oeil -> couche 1-2-4(en plus de 3) devient visibles et icone devient oeil barré. clic sur oeil barré -> toute les couches deviennent non-visibles et icon oeil devient oeil noir |
Moi je vote pour la proposition A, ça me semble plus clair pour l,utilisateur et la plupart des outils fonctionnent comme ça... Si c'est un oeil pour rendre visible, ça sert à rendre les couches visibles! Sinon c'est plutôt un outil pour inverser la visibilité de chaque couche individuellement, il faudrait changer l'icône. Est-ce que cet outil devrait être présent en tout temps, pas seulement en mode sélection multiple? Ça va être mêlant avec l'oeil qui est déjà présent quand la sélection multiple n'est pas activée, qui a un comportement différent. Pourquoi ce bouton ne servirait pas aussi à rendre facilement toutes les couches visibles ou à masquer toutes les couches d'un coup? Ça serait utile selon moi. Le bouton actuel est plutôt un filtre et je ne trouve pas que l'icône représente bien ce qu'il fait. |
Faut juste faire attention a la performance lorsque l'ont ouvre toutes les couches. |
En effet, mais c'est le même problème si les utilisateurs ajoutent un grand nombre de couches. Ils pourraient tout sélectionner aussi dans l'outil de sélection multiple. |
Imagine si en plus si tu as l'option ouvrir les légendes automatiques à l'ouverture de la couche. |
Un avis indiquant que lorsque tu as plus de 15 couches, ça risque d,affecter grandement la performance.....? |
Avec un "Continuer Oui/Non" |
Ça dépend des couches? Les couches plus légères, il pourrait y en avoir un plus grand nombre sans affecter la performance (ou moins l,affecter) ou c'est vraiment le nombre qui est en cause? |
Il faudrait tester. |
Oui avant de mettre un avertissement de la sorte il faudrait voir. Si la navigation est plus lente l'utilisateur s'en rend bien compte par lui-même, il peut retirer ou masquer des couches facilement à ce moment avec les outils qui sont en place. Bref je ne priverais pas les utilisateurs de ces fonctionnalités pour autant |
Idée: Quand tu as coché plus de X couche(a determiner) par exemple 10, l'option de oeil se désactive(grisée) et le tooltip desssus indique : "l'option d'afficher toute les couches ne peut fonctionner avec plus de 10 couches" |
Il ne faudrait pas que ça empêche des les masquer par contre... Je ne sais pas pour moi c'est vraiment à l'utilisateur de contrôler ça, on s'entend qu'un navigateur carto web aura toujours une limite où ça devient trop lourd, que ce soit 10, 20, 30 ou 50 couches... et ça peut probablement dépendre de la nature des couches, si j'ai 20 couches pas très lourde peut-être que tout va bien en fait! |
Mais concrètement, pour un wms,il n'y a pas de différence entre une couche lourde ou pas... Bref, je ne vois pas le lien lourd/pas lourd Est-ce lié a un problème de cache... ? |
Je ne sais pas qu'elle est le % des gens, surtout très grand public, qui seront 'compréhensifs' sur le fait que l'outil grafigne quand on en coche trop... Certains diront-> Cet outil ne va pas bien! Je pense qu'on doit limiter les comportements de l'utilisateur qui peuvent être problématique pour l'outil. On ne doit pas donner la possibilité à l'utilisateur de faire 'planter' l'outil. |
L'avis ferai la "job" a mon avis. |
Effectivement, avoir des problèmes avec 10 couches c'est quand même grave! Ça me surprend un peu |
Dans l'exemple précédent, c'est près de 40 couches. |
faudrait trouver le seuil acceptable. |
Dans ce cas à mon sens, c'est quand même extrême comme utilisation, c'est à nous aussi de ne pas proposer des contextes à 40 couches? Bref je ne limiterais pas l,outil de visibilité à ce moment. L'avertissement pourrait faire, ma seule crainte c'est de recevoir souvent cet avertissement et de faire mal paraître l'application alors que la performance pourrait être quand même "acceptable". Si c'est 15 couches le max il me semble qu'on a un problème! Mais si c'est 30-40 c'est quand même pas mal moins pire |
Le problème, c'est l'ajout de couche par le catalogue. |
Par rapport à la "lourdeur", je croyais qu'un WMS avec beaucoup de détails l'image est plus lourde donc le rendu pourrait être plus difficile par le fureteur? Mais peut-être que ce n'est pas la cas! Dans ton exemple aussi, il pas mal de WMS qui prennent plus de 2 secondes à répondre (jusqu'à 6), c'est quand même long |
Même une fois chargées, le pan reste difficile. ca doit pas être lié au temps d'attente de la réponse |
Idée : pour garder cela simple et agile on pourrait faire un premier dévellopement qui répond au besoin initiale On voit à l'usage limitation, seuil, disposition et si nécessaire, a ce moment, on demandera une amélioration/modification |
Je seconde, j'ai ouvert un autre issue concernant ce qu'on fait si trop de de couches sont ajoutées sur la carte #493 |
la performance dépend aussi de ton poste, moi je n'ai pas de problème avec le contexte cité ! À part que c'est long à afficher les wms mais ce n'est pas à cause de IGO ! |
@infra-geo-ouverte/depot-central-igo Vous pensez quoi de ce correctif. Il permet d'appeler seulement l'extent de la map plutot que 1.5x sa taille. Sa semble régler le problème. Vous devez faire le test sur un écran de grande taille + 1700px de largeur |
* fix(language) Get/set language is desynchronized *
Please tell us about your environment:
Igo Version:
Node:
The text was updated successfully, but these errors were encountered: