search

assemblee-virtuelle / archipelago

Fostering interconnections between communities by creating synergies between their platforms
Apache License 2.0
14 stars 6 forks source link

Adding Tree function to Topic #134

Closed BastienSig closed 9 months ago

BastienSig commented 10 months ago

Rajout des thèmes arborescent première. @simonLouvet Ici gestion de la liste des thèmes en arborescence, ajout du thème parents dans un thèmes enfant, ajout d'un thème dans une ressource en arborescence.

srosset81 commented 10 months ago

Je laisse @mguihal faire le review, si c'est OK pour lui.

srosset81 commented 9 months ago

On pourrait utiliser un composant comme https://github.com/mikepricedev/mui-tree-select ou https://github.com/dowjones/react-dropdown-tree-select qui font directement le job.

L'UX de https://github.com/mikepricedev/mui-tree-select est intéressante, ça vaudrait la peine d'essayer avec des cas concrets, surtout si le composant peut être utilisé tel quel. Par contre je me demande si https://github.com/dowjones/react-dropdown-tree-select fonctionnerait bien en mobile. A tester...

BastienSig commented 9 months ago

Je ne bloque pas s'il y a nécessité de merger cette feature, mais il y a plein de petits détails qui me dérangent. Est-ce qu'il est prévu de repasser dessus pour en faire une fonctionnalité plus aboutie ?

D'une manière générale, je ne suis pas fan du double input (autocomplete + modal avec treeView) lors de la sélection du thème ou du parent, c'est confus pour l'utilisateur. On pourrait utiliser un composant comme https://github.com/mikepricedev/mui-tree-select ou https://github.com/dowjones/react-dropdown-tree-select qui font directement le job.

Je vais faire un test avec mui-tree-select voir ce que ça donne =)

BastienSig commented 9 months ago

J'ai fais une nouvelle update avec le composant https://github.com/mikepricedev/mui-tree-select proposé par @mguihal. Il reste peut être des coquilles que je n'ai pas vue. Mais ça fonctionne bien, Simon m'a aidé a modifié un peut le comportement du composant pour pouvoir sélectionner les parents et non que les enfants.

@srosset81 @simonLouvet

srosset81 commented 9 months ago

@BastienSig Pour des nouveaux composants comme ça, ce serait chouette d'inclure une vidéo dans la description de la PR, ça évite d'avoir à tout lancer en local et on peut quand même donner un feedback en fonction du résultat.