feat: Nouvelle page pour désactiver le son/vibreur + Page du choix des thèmes native

[Kgeek33]

🚀 Nouvelle Pull Request

Proposez vos modifications pour améliorer Papillon

Informations importantes

Merci de vous référer à la documentation sur la contribution si vous avez des questions à propos des pull requests (https://gitbook.getpapillon.xyz/organisation/outils-internes/github)

Checklist d'avant pull request

Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].

• Vous avez testé de build le projet avec vos modifications et ce build a réussi
• Vous respectez les conventions de codage et de nommage du projet
• Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
• Cette pull request n'est pas un duplicata d'une autre
• Cette pull request est prête à être revue (review) et fusionnée (merge)
• Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
• Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
• Les détails des changements ont été décrits ci-dessous
• Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

• Une nouvelle page dans les paramètres a été crée : "Son et vibrations". Elle permet de d'activer ou non le son et les vibrations dans l'app

Note

La valeur sera toujours sur true si l'utilisateur ne change pas ce paramètre/quand il installe l'app pour la 1ère fois

• La page pour changer le thème de l'application est désormais native ! Ca signifie que quand le paramètre est changé, le thème est automatiquement appliqué (plutôt que redémarrer l'app comme l'avait signalé @Gabriel29306)
• Pour appliquer tout ça, un hook a été appliqué qui utilise createContext de React Native pour une synchronisation Papillon <=> AsyncStorage
• Optimisation du code

Issues en lien

• Close [Enhancement]: avoir un paramètre pour désactiver les vibrations. #339
• Close [Bug]: Texte invisible dans les paramètres #597

Informations supplémentaires

Warning

Désolé si les vidéos ne s'affichent pas dans le tableau 😅

Fonctionnalité	Photo/Vidéo
Changement du thème	https://github.com/user-attachments/assets/9db2c704-bc76-40e6-ab62-bb28289c24c1
Page "Son et vibrations"
Changement du paramètre du son	https://github.com/user-attachments/assets/76fea8d1-3e39-4510-b092-44d37365396e

[ecnivtwelve]

LGTM 👍🏽

[Nicolasfr340]

Peut-être aussi pouvoir régler l'intensité de l'haptic, je crois c'est possible (jsp si c'est juste sur certains téléphones par contre)

[Kgeek33]

Peut-être aussi pouvoir régler l'intensité de l'haptic, je crois c'est possible (jsp si c'est juste sur certains téléphones par contre)

Ça c'est pas possible à intégrer et à ma connaissance, seul la marque Samsung permet de changer le type de vibration

[godetremy]

Note

Cette review est le résultat de la concertation de toute l'équipe Papillon.

Modification à apporter

Renommer les réglages vers "Accessibilité"

Il serait intéressant de rassembler "mode d'affichage" et "son et vibration" au sein d'un onglet accessibilité dans la catégorie "avancé".

Rendre ton code plus simple

Simplifie tes conditions avec un wrapper

[godetremy]

Note

Cette review est le résultat de la concertation de toute l'équipe Papillon.

- Pas de modifications notable -

[Kgeek33]

Oui je me suis principalement concentré sur la pr #606, je fix ça ce soir ou demain

[Kgeek33]

@godetremy c'est bon, j'ai apporté les modifications comme demandé à une exception :

Selon moi, regrouper tout dans un onglet Accessibilité peut certes être une bonne idée, mais cela peut porter à confusion l'utilisateur, cela pourrait faire penser par exemple que l'app est compatible avec TalkBack / VoiceOver (alors que ce n'est peu le cas)
De plus, laisser séparer les paramètres peut sembler plus logique pour l'utilisateur, car les téléphones font la même chose dans les paramètres systèmes

Je suis toujours ouvert à tes suggestions :)