-
Notifications
You must be signed in to change notification settings - Fork 24
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
Performance client (carto) #55
Comments
C'est un problème qui apparait avec un grand nombre de site. Il y a tout une politique de gestion de l'affichage des sites à définir
pour ce problème particulier je ne ferai pas de configuration spécifique à un module, mais ça pourrait être intéressant dans d'autres cas ?? |
Salut, gn_module_monitoring/frontend/app/components/monitoring-map/monitoring-map.component.ts Lines 47 to 79 in 4f7d1ec
SAUF que, au moment du chargement de la page, la carte est d'abord générée avec le style par défaut de Leaflet (avec la transparence donc), PUIS quelques secondes après, le style écrit dans monitoring-map.component.ts est appliqué. Le fait d'avoir commencé par appliquer le style par défaut fait freezer inutilement le client. Mais peut-être que c'est un problème à traiter au niveau du coeur de GN (pnx-monitoring-map ? ) |
Bonjour, Je me permets de relancer ce ticket. Est-ce que des développements sont en cours ou prévus au niveau du module monitoring ? |
Salut,
J'ai l'impression qu'en supprimant la transparence par défaut des points, on améliore assez significativement la performance côté client (testé sur ~4000 points).
gn_module_monitoring/frontend/app/components/monitoring-map/monitoring-map.component.ts
Lines 47 to 79 in 4f7d1ec
Petit problème, la couche est rendue dans un premier temps avec un style par défaut (bleu+transparence), puis le style définit dans map.component est appliquée dans un second temps. Ceci freeze (inutilement ?) le client pendant un petit moment.
Est-il possible de déporter la définition du style de la map au niveau de la configuration de chaque sous-module ?
A+
The text was updated successfully, but these errors were encountered: