PapillonApp / Papillon

Une alternative open source aux applis de vie scolaire. L'allié de tous les étudiants. Le futur de l'éducation numérique libre et ouverte.
https://papillon.bzh
GNU General Public License v3.0
114 stars 42 forks source link

feat: Équité entre iOS et Android #378

Open Kgeek33 opened 1 week ago

Kgeek33 commented 1 week ago

🚀 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].

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

Issues en lien

Informations supplémentaires

les captures arrivent bientôt

Gabriel29306 commented 1 week ago

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 commented 1 week ago

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 commented 1 week ago

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 commented 1 week ago

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 commented 1 week ago

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 commented 1 week ago

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