Feature/modul 582 arbo multinode select

[DDurandDTI]

@ulaval/modul-components

PR Checklist

• Breaking Changes

Trees without icons do not exist anymore and default display mode is without files. As such, to conserve current icons (with files and folders), tree component needs use-file-icons prop to be true.

--> Ajouter :use-files-icons="true" à l'arbre pour afficher les dossiers. Par défaut, on a maintenant le plus et le moins.

Selection modes for tree are now words:

• 0 becomes none,
• 1 becomes single,
• 2 becomes multiple

--> Pour le single, changer :selection-mode="1" pour :selection-mode="single"

• Provide a small description of the changes introduced by this PR

• New checkbox states

• Moved TreeNode and TreeIcon to Tree Submodule

• Finished the multiple nodes selection for the Tree component

• Added 5 new types of possible Tree:

1. Simple Multiple Nodes,
2. Checkbox AutoSelect Multiple Node,
3. Parent Checkbox AutoSelect Multiple Node,
4. Checkbox Button AutoSelect Multiple Node,
5. Multiple Node + Checkbox Without AutoSelect.

• Changed Icon component to allow it to print name as class (required for CSS selection)

• Added new tests for every modified components

• Added sandbox elements for every modified components

• More stuff; requirements for the components evolved more than Son Goku ever did.

• Include links to issues
  https://jira.dti.ulaval.ca/browse/MODUL-582
  https://jira.dti.ulaval.ca/browse/MODUL-542

• Openshift deployment requested

[Netnix]

C'est vraiment bon ! J'aurais préféré que ce soit divisé en quelques PR plus petites (une par type d'arbre?).

[Netnix]

Mais n'oublies pas de régler les conflits.

[jfdion]

Pour aider à la compréhension du sandbox, lorsqu'il s'agit d'un élément sans enfant que l'on peut sélectionner, j'utiliserais leaf/feuille plutôt que node. Sinon c'est très bien!

[DDurandDTI]

@Netnix Je vais gérer les conflits comme un arbitre. Genre Anne-France Goldwater. Je suis d'accord que les merges auraient pu être mieux découpés. La prochaine fois, ça va être plus mieux :).

J'ai changé le terme node pour leaf dans le sandbox aux endroits qui concernent le multiple node.

Merci des retours.

[chuckmah]

Good job 👍 il reste quelques changements mineurs a corrigé

[setur52]

@chuckmah @DDurandDTI est-ce que c'est proche d'être dans develop, je vais avoir des bonifications à faire sur ce composant.

[chuckmah]

@setur52 Prevu pour la 73 (mecredi prochain) si les reviews et le QA a @jipigi

Sinon tu peux toujours brancher à partir de cette branche pour tes changements mais tu risques des conflits avec ce qui se passe dans cette branche, donc il faudra te tenir a jours avec cette branche

ps. j'attenderais aussi que develop soit mergé avec cette branche, elle est un tantinet out-dated

[DDurandDTI]

@setur52 @chuckmah

Je suis en train d'appliquer les recommandations du code review. Je vais mettre à jour la PR bientôt

[DDurandDTI]

@chuckmah

Désolé pour le délai. J'ai ajusté pas mal de choses. Le plus gros changement est que tous les ajouts de noeuds passent par l'arbre, donc j'ai dû refaire tous mes tests unitaires. J'ai retiré des props, j'ai ajouté des getters, j'ai mis à jour la branche avec dev et j'ai géré les conflits. J'ai ajouté des breaking changes à la PR (je ne savais pas que je pouvais briser les choses qui me bloquaient déjà présents dans le code).

Tout est plus propre, mais ça demande d'autres vérifications. Merci de vos patiences de Jedi sans robe.

[jipigi]

@DDurandDTI Merci pour tout ce travail dans l'arbo, en effet, San Goku fait pâle figure devant toutes ces amélioration :)
Je vais créer un billet pour voir comment on fait évoluer le WithButtonAutoSelect, qui pour l'instant est dans un état intermédiaire.