Skip to content
Snippets Groups Projects

Feature/node image

Merged LIOTIER MARION requested to merge feature/nodeImage into develop

feature pour afficher des images dans les noeuds : images rondes ou carrés

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 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 !!

    • ok c'est maintenant dans /public/images

    • image et imageRect -> renvoie le style pour afficher un rectangle ; imageCircle -> renvoie le style pour un cercle

    • ok

    • ok

    • non l'ajout de l'image se fait à la construction de l'objet network

    • oui ça fonctionne avec les autres réseaux

    Merci !

  • LIOTIER MARION added 5 commits

    added 5 commits

    Compare with previous version

  • Jean-Clement Gallardo approved this merge request

    approved this merge request

  • mentioned in commit acc3d631

Please register or sign in to reply
Loading