fix(map): resolve marker latency, issue 360 #416
Conversation
| @@ -65,10 +65,10 @@ const Property = ({ | |||
| {loading ? ( | |||
| <Skeleton variant="text" width="100%" /> | |||
| ) : ( | |||
| <Box display="flex" flexDirection="column"> | |||
| <div style={{ display: 'flex', flexDirection: 'column' }}> | |||
There was a problem hiding this comment.
On pourrait rajouter un commentaire pour expliquer pourquoi on passe un div peut-être ? J'imagine que la Box répétée des milliers de fois étaient trop lourde.
J'imagine que la modif venait de moi, je prends note du problème de performance que ça peut induire, désolé.
There was a problem hiding this comment.
Il doit y avoir un bug avec le composant "Box" car normalement cela ne devrait pas provoquer de ralentissement... Oui, mettre un commentaire pour expliquer la raison du div ici serait bien :)
There was a problem hiding this comment.
J'ai trouvé ce commentaire à propos des performances de Box : mui/material-ui#21657 (comment)
A partir de la v5 de MUI (on est toujours en v4 sur Grottocenter), ils l'ont optimisé et elle s'affiche 5 fois plus vite.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://orange-rock-0d4f87503-416.westeurope.azurestaticapps.net |
|
J'ai supprimé sans faire expres le commit de @bsoufflet |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://orange-rock-0d4f87503-416.westeurope.azurestaticapps.net |
fix #360 - add a comment on map issue - update default map location of map - add preferCanvas prop - prevent zooming too much out (for better performance)
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://orange-rock-0d4f87503-416.westeurope.azurestaticapps.net |
Resolve issue #360
Replace MUI
Boxcomponent todiv