feat: Papillon 100% compatible avec les tablettes

[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

• Changement de l'orientation sur app.json pour permettre le mode paysage sur Android !
• Création d'un hook qui détecte le mode portrait/paysage
• Utilisation de ce hook dans la majorité des pages
• Fix de l'emploi du temps inutilisable sur les tablettes
• Optimisation des performances

Informations supplémentaires

Note

C'est pas si lent (excepté la page des devoirs), c'est ma tablette qui en pouvait plus 🤣

Screenrecorder-2024-12-22-14-51-57-206.mp4

Issues en lien

• Closes [Bug]: Problème affichage EDT iPad #337
• Closes [Bug] App: redémarrage de l'app lors de la rotation de l'écran  #251
• Closes [Bug]: L’emploi du temps ne s’affiche pas correctement sur iPad #458
• Closes [Bug]: Affichage de l’EDT sur Tablette #593

[ecnivtwelve]

LGTM après tests

[ecnivtwelve]

LGTM 👍🏽

au passage, si ta PR n'est pas prête, met la en draft, c'est plutôt chiant d'avoir a re-review régulièrement

[Kgeek33]

LGTM 👍🏽

au passage, si ta PR n'est pas prête, met la en draft, c'est plutôt chiant d'avoir a re-review régulièrement

si, elle est prête, c'est juste que j'ai des conflits régulièrement et ça demande ta review à chaque fois que je résous les conflits

[ecnivtwelve]

désolé sur ce point, ma PR va possiblement casser un peu la tienne --> #556

[Kgeek33]

désolé sur ce point, ma PR va possiblement casser un peu la tienne --> #556

je vais review pour voir

[Nicolasfr340]

@Kgeek33 t'as aussi fix ça? https://github.com/user-attachments/assets/c60509c2-dab9-4c97-902c-1879542ccf7e

Sur iPad Mini 6, iPadOS 18.1, app native AppStore

[Kgeek33]

@Kgeek33 t'as aussi fix ça? https://github.com/user-attachments/assets/c60509c2-dab9-4c97-902c-1879542ccf7e

Sur iPad Mini 6, iPadOS 18.1, app native AppStore

là, faut aller voir ma pr #483 :)

[Nicolasfr340]

Bv la 338

[godetremy]

Note

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

Résultat du test sur Expo Go

Beaucoup de petit fails, mais le résultat est prometteurs.

Erreur n°1 - Texte trop loins

Sur tes pages, tu as tout rejoins au centre (ce qui est tout à ton avantage) mais seul ce petit texte se retrouve exclu...

Erreur n°2 - J'ai pas pu me log

Les listes sont toute recroquevillée sur elles même ce qui cause pas mal de problème d'interface... Voici plusieurs endroits ou l'on a vu le problèmes :





Juste à modifier le composant de la liste !

Erreur n°3 - Le SlideOver ne fonctionne pas

On a vu cette écran, mais c'est vrai que peux comprendront l'utilité... Donc il faudrait l'afficher uniquement si le format affiche la tabbar.

Erreur n°4 - La modal bieeeeeeeeeeeeeen longue

Le composant de Vince n'est pas réduit ce qui cause de longueeeeeeee modal. Une width max avec un centrage serait merveilleux !

Erreur n°5 - Qu'est-ce que tu m'as dit ?

La bulle lors de la configuration des services de restaurations n'est tout simplement pas visible.

Une fois tout cela réglé, une nouvelle vérification sera effectué puis merge. ✌️

[Kgeek33]

Ok merci à l'équipe de votre review, j'vais essayer de corriger ça !

[Nicolasfr340]

Quand on clique sur notre nom prénom, il y est 2 fois (pas ouf) voir 3 avec l’accueil