Feature/node image
feature pour afficher des images dans les noeuds : images rondes ou carrés
Merge request reports
Activity
Filter activity
requested review from @jean-clement.gallardo
Plusieurs remarques global sur les ajouts :
- Éviter de mettre des images pour les tests dans /src/assets. Ce dossier est destiné à la gestion interne du VizCore. Si tu souhaites les garder pour des tests futurs, il vaut mieux les mettre dans le dossier public.
- Pour le UsePolygonShape, pourquoi mettre plusieurs label (image, imageRect et imageCircle) pour le même rendu ?
- Dans ton ImageComponent, au niveau du style, il vaut mieux mettre (pour l'import) ../assets/style.css. Ça évite des problèmes de style au moment du build
- Pour les props du ImageComponent, pour le style, il n'y a pas besoin de mettre une valeur par défaut. Via les fonctions de récupérations (getNodeStyle), on gère déjà la possibilité d'un style vide. Sauf si tu as besoin d'en avoir un en particulier.
Petites questions :
- Tu penses que c'est possible de faire un import d'image pour les nœuds en direct ? Pour l'instant, si j'ai bien compris le fonctionnement, tu dois mettre directement l'URL + le shape dans le JSONGraph à l'import. Du coup, est-ce que se serait possible d'arriver avec un réseau "classique" et d'y mettre des images en cours de route ?
- Est-ce que tu as vérifié que le code fonctionne correctement sans les images (avec un réseau classique).
À part tous ça, tout me semble bien !!
mentioned in commit acc3d631
Please register or sign in to reply