fix: be able to use sidemenu with multiple 3 levels submenus

[KGALLET]

Fix the issue #319

Not sure if i did all the requirements in order to contribute to the repo, feel free to ask me further dev / informations.

[garronej]

Salut @KGALLET,

Merci pour la contribution.
Le DIFF est un peut gros, c'est difficile de voir ce que tu as changer vu que tu as refactorer.
S'il te plais rebase avec un diff minimum qui ne traitre que le bug en question.

[KGALLET]

Salut @garronej,

Je suis repassé et j'ai modifié le commit. C'est beaucoup plus visible en effet sur la diff maintenant. Juste un soucis au niveau de la "key" props.
Désolé d'avoir refacto, c'est de la façon dont j'ai procédé pour trouver itérativement et corriger le bug.

Pour info, j'ai retesté sur le playground avec CRA, vite, et NextJS (app + pages)
Pour reproduire le bug, tu peux juste copy paste le bout de code dans l'issue #319 dans le playground et essayer de dérouler tous les menus à 3 niveaux

Dis moi si c'est ok pour toi maintenant
A dispo

[KGALLET]

@garronej
Hello,
any updates on this ? :)

[garronej]

Désolé d'avoir manqué la modification.
Un grand merci d'avoir pris le temps non seulement de diagnostiquer le bug, mais aussi de proposer un correctif.

Le DIFF pourrait littéralement se résumer à une ligne. Ici, tu as variabilisé isExpended, ce qui n'a pas de lien direct avec la PR, et cela pourrait même suggérer que c'est un état contrôlé, alors qu'il s'agit en réalité de la valeur par défaut lorsque le composant est monté. La sémentique est trompeuse.

Peux-tu faire un rebase avec juste la ligne concernée ? Je te promets de merge et de release immédiatement après.

[KGALLET]

@garronej
Pas de soucis !

c'est fait :)

[KGALLET]

Thx @garronej