Add the event command library component#677
Conversation
There was a problem hiding this comment.
Hello ! J'ai pu tester ta PR et c'est vraiment top ! 😄
Je mets ici quelques remarques et bugs que j'ai trouvé au fur et à mesure.
Important
On en parlera lors du point design, lors d'une démo avec Walven, ce commentaire n'appelle pas une réponse ici pour le moment.
Remarques
- Après usage, c'est la deuxième fois que je me surprends à vouloir drag & drop une catégorie sur le canva, alors qu'il me faut aller dans une catégorie pour obtenir la liste des commandes à drag & drop. Aelysya a eu la même problématique, je pense qu'il faudra qu'on revoit la partie catégorie pour qu'on ne soit pas tenté de drag & drop.
- La librairie de base rend la liaison entre deux nodes assez pénible. Il faudra vraiment qu'on soit vigilant sur cette partie lors de la réalisation du design car le branchement entre les nodes doit être agréable à réaliser, et surtout ne pas demander à l'utilisateur de maîtriser son curseur de façon abusive.
- On peut relier une node à elle-même en reliant les deux entrées. Mais on ne peut pas connecter 1000 nodes. Prenez un chewing-gum Emile.
Bugs
- Problème avec l'icône d'une catégorie
Lorsque le nom d'une catégorie fait 3 lignes, l'icône est écrasée en hauteur.
- Le nom de la commande dépasse de la boite de commande quand le texte est trop long
-
Lorsqu'on sélectionne une commande et qu'on la maintien en drag & drop au dessus du canva, et qu'on fait du gauche/droite rapide avec la node "Shadow" affichée, le curseur semble s'emballer et passer de drag & drop à impossible de drag & drop.
a. Si on drop la commande au moment où l'icône est en interdit, la node reste "Shadow", et ajouter une nouvelle commande sur le canva viendra la supprimer. -
Lorsqu'on sélectionne une commande et qu'on la maintien en drag & drop au dessus du canva, et qu'on fait des mouvements circulaires lentement avec le curseur, les deux composants que sont le visuel de la boite de commande (carré provenant de la liste dans la bibliothèque) et la node "Shadow" bougent de façon désynchronisés.
-
Lorsqu'on manipule les nodes, à certains moments, la nouvelle commande drag & drop depuis la bibliothèque possède déjà une liaison avec une node existance dès son apparition. Sans doute liée à un état actif / focus / selected d'une node existante avant de drag & drop. Le style actuel ne permet pas de bien voir, mais on règlera ça avec un premier design.
AntoinePoree
left a comment
There was a problem hiding this comment.
Dans l'ensemble, c'est good malgré les points qu'on a vu avec Aerun, et je me permet de revoir la création de données, pour préparer le plus tard aussi. Je sais que c'est embétant, mais plutôt on s'y prend mieux c'est.
Si besoin d'aide, n'hésite pas, je peux m'occuper de deux trois sujets !
|
Tout ce qui concerne les nodes pour le moment c'est expérimental. J'ai ajouté très rapidement la node shadow car j'ai eu une idée pour l'ajouter, qui a plutôt bien fonctionné, mais je n'ai pas testé les cas limites (parce que la partie node reste du poc pour le moment) Pour les problèmes de textes qui débordent, c'est la raison pour laquelle la PR est en draft. (je cite : In draft because I need to fix the text overflow.) :p Quand j'ai ajouté le français pendant que j'écrivais la description de la PR, j'ai vu qu'il y avait des problèmes d'où le draft. ^^' |
|
Changements apportés :
Correction non réalisée de la node shadow car je n'ai pas trouvé cmt faire (mais ça reste expérimental on verra plus tard) La PR n'est plus en draft :) |
AerunDev
left a comment
There was a problem hiding this comment.
Il faudra juste gérer les conflits i18n, et on sera bon pour cette première PR concernant la bibliothèque des commandes. Bien joué ! 😄
83c445d to
d3e477b
Compare
…Context to manage a vertical text-overflow
7f341e9 to
cc148fe
Compare
Description
This PR implements the event command library. For the moment, the editor is available in the event editor poc page.
The style of the nodes is still the default. The command type is transmitted correctly but for the moment only the text change. So it is theoretically possible to display the correct node command in react flow.
The user can:
The component has been implemented in such a way as to make it easy to add new categories or commands. (we can add this in the documentation)
In the files
src\models\entities\event\command.tsandsrc\models\entities\event\category.ts, we can add a command, a category and a command in a category.In the file
src\views\components\event\EventCategoryIcon.tsx, in the constIconsFromCategorywe can link an icon and a color to a category.Don't forget to add in i18n files the translation. The rules for the keys:
event_category_{category_name}event_command_{command_name}event_command_{command_name}_helperImprovement:
type EventCategoryColor = 'violet' | 'blue' ...)Closes #428
Tests to perform
Screenshot
Pokemon.Studio.2025-12-08.21-03-49.mp4