♻️ Improve checkbox component state

[mcampourcy]

Note :

Le changement pose quelques soucis, notamment parce que la checkbox est utilisée par le composant multi-select et un filter (qui appelle multi select qui appelle checkbox). On se retrouve avec un enchainement d'actions au on-change.

Est-ce que ce ne serait pas l'occasion de passer aux composants Ember Checkbox ?

🎄 Problème

Il y a un décalage entre l'état réel du composant et le paramètre isChecked.
Si on passe le isChecked à true, puis qu'on clique sur la checkbox, le isChecked reste à true au lieu de repasser à false.

🎁 Proposition

Modifier le paramètre au clic.

🎅 Pour tester

Cliquer sur la checkbox, vérifier que tout est OK

[pix-bot-github]

Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr537.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr537/environment

[xav-car]

Est-ce que ce ne serait pas l'occasion de passer aux composants Ember Checkbox ?

Je dirais non, on ne veut pas des composants natif Ember puisqu'en V5 et futur ils sont censé disparaître ( même si il serait disponible via un package legacy, mais ça ne semble pas forcément le bon move )

[yannbertrand]

À finaliser, ouvert à la contrib. Voir https://1024pix.slack.com/archives/C0149DV4873/p1713256734589479

[mcampourcy]

Je proposerais bien de fermer cette PR : le composant a connu pas mal de changements dernièrement, il faudrait voir si la question de l'amélioration se pose toujours