assemblee-virtuelle / archipelago

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

[Minor] Improve TreeMenu #131

Closed mguihal closed 12 months ago

mguihal commented 1 year ago

Hello,

Voici quelques améliorations concernant le menu latéral :

1/ Meilleure utilisation des éléments de React-admin pour prendre en compte toute la largeur de l'espace menu pour les items. J'ai aussi supprimé les props qui ne sont plus implémentés dans le code de react-admin, et nettoyé les styles custom rajoutés devenus inutiles.

Avant :

Capture d’écran 2023-07-27 à 23 28 04

Après :

Capture d’écran 2023-07-27 à 23 28 16

2/ Meilleure intégration des submenus quand le menu est replié, comme recommandé dans les exemples de React-admin.

Avant :

Capture d’écran 2023-07-27 à 23 28 32

Après :

Capture d’écran 2023-07-27 à 23 28 47

3/ Correction de la position des tooltips

Avant :

Capture d’écran 2023-07-27 à 23 29 41

Après :

Capture d’écran 2023-07-27 à 23 29 56

4/ Ajout d'un indicatif de sélection de l'item actif (trait de 3px de la couleur primaire du thème)

Capture d’écran 2023-07-27 à 23 28 16

5/ Depuis le passage à la v1.0.0, un bouton "Déconnexion" a été rajouté dans le menu, dans la version mobile seulement, mais celui-ci apparaissait tout le temps, même si on n'était pas connecté. J'ai modifié pour ajouter les boutons d'inscription et de login à la place dans ce cas. Vu que les boutons d'inscription/login/logout sont toujours disponible dans l'appBar en haut à droite dans la version mobile, il est aussi possible de ne pas rajouter ces boutons dans le menu...

Avant :

Capture d’écran 2023-07-27 à 23 30 34

Après :

Capture d’écran 2023-07-27 à 23 43 48 Capture d’écran 2023-07-27 à 23 44 34