feat: Équité entre iOS et Android

[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

Note

Cette PR n'est pas encore prête, en cours de travail. Mais est listé ce qui est prévu, peut évoluer

• Alert.alert pour iOS est supprimée => utilisation de useAlert() d'Android
• Convertion Modal => SlideModal pour permettre de slider des modals comme iOS y compris sur Android
• Suppression ActivityIndicator => PapillonSpinner
• Fix de l'affichage des émojis non centré

Issues en lien

• Closes [Bug]: Fenêtre lorsqu'on clique sur un devoir, information et une note trop haute #233
• Closes [Enhancement]: Center subject's icon #256
• Closes [Feature]: Ajouter un second provider useAlert dans la navigation view des Settings sous iOS #157
• Closes [Feature]: Crée un composant PapillonContextMenu #169
• Closes [Enhancement]: Style message ICal (Non Urgent) #463

Informations supplémentaires

les captures arrivent bientôt

[Gabriel29306]

Le Alert.alert est nécessaire dans certains cas dans iOS, spécialement pour la page des paramètres. Le moyen pour résoudre ce problème est de rajouter le contexte nécessaire pour pouvoir utiliser le useAlert.

[Kgeek33]

Oui c'était juste une idée que j'ai eu, faut que je teste maintenant mais je me doute que dans certains endroits, ça ne va pas fonctionner

[tom-theret]

Je suis vraiment pas d'accord avec le concept des modals, c'est ce qui fait le charme d'iOS et c'est surtout de la cohérence dans le système.

Concernant les alerts ça serait bien de porter les bottomSheetAlert sur toute les page mais il faut un provider sur iOS.

[Kgeek33]

Bien sur je suis d'accord avec toi. L'idée vient que depuis une version, les modals sur Android contient une barre, qui fait (en tout cas pour moi), qu'on a le réflexe de faire slider la modal, mais uniquement sur ios.
Je me renseignerai pour les alerts

[Gabriel29306]

Si on veut faire un truc un peu plus simple entre Android et iOS:
https://reactnative.dev/docs/platform-specific-code#platform-specific-extensions

[Kgeek33]

Oui je l'ai fait pour la page d'accueil, un revert de la version 7.2 pour Android et la page d'accueil actuelle (7.3) pour ios

[Kgeek33]

Bon je voulais attendre que la pr pour la compatibilité expo 52 soit merged, mais ça a pas l'air de bouger...

[Kgeek33]

j'attends qu'on soit sur expo 52 pour pouvoir poursuivre cette pr
close pour le moment