search

dropy-project / front

5 stars 0 forks source link

Energy popup UX adjustments #420

Closed celian-rib closed 1 year ago

celian-rib commented 1 year ago

les informations sur l’énergie disparaissent trop vite, faire un système de clic en dehors du pop up pour le faire disparaître

enzo-mourany commented 1 year ago

Pour la fonctionnalité ‘Cliquer en dehors de la popup la ferme‘ , j'avais pensé à défini une props 'render' mais ce n'est pas vraiment l'idéal.

Est-ce que RN gesture handler permet de voir si on click en dehors du component ?

celian-rib commented 1 year ago

@enzo-mourany j'ai jamais fais, si tu peux pas le faire avec rn gesture handler il y a surrement d'autres librairies, a toi de chercher ;)

enzo-mourany commented 1 year ago

J'ai trouvé react-outside-click-handler

gfroidcourt commented 1 year ago

En général faut éviter d'utiliser des packages quand on peut faire sans. Ça inclus beaucoup de dépendances et de sous dépendances donc de potentielles failles de sécurité ainsi que beaucoup de mise à jour de packages à faire. Si tu ne peux pas faire sans, pas de soucis mais autant éviter un maximum