Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Papillon 100% compatible avec les tablettes #533

Open
wants to merge 90 commits into
base: main
Choose a base branch
from

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Dec 22, 2024

🚀 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
  • Affichage uniquement d'un <AccountSwitcher> en mode paysage

Informations supplémentaires

Note

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

1734875857232

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

1737247154571
1737247154573

Issues en lien

Kgeek33 and others added 30 commits October 29, 2024 17:59
@Kgeek33 Kgeek33 requested a review from godetremy as a code owner January 19, 2025 00:36
@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Jan 19, 2025

@godetremy c'est bon, je viens de fix tous les points signalés exceptés...

Erreur n°1

Nouveau rendu :
1737247154553

Erreur n°2

Désolé, mais je n'arrive pas à reproduire le bug sur Android et je n'ai pas d'iPad pour tester...

Erreur n°3

Cette page n'est plus accessible depuis les paramètres en mode paysage (en mode portrait, ça s'affiche bien)
1737247154566

Erreur n°4

Je m'en étais pas rendu compte, nouveau rendu :
1737247154558

Erreur n°5

Effectivement, elle posait problème cette page, corrigé !
1737247154562

Erreur de @Nicolasfr340

Corrigé, affiche le nom du compte une seule fois sur la NavBar de la tablette, aucun impact en mode portrait :

Paysage :
1737247154573

Portrait :
1737247154571

@codeuriii
Copy link
Contributor

Est ce que on peut faire en sorte que si on est sur tablette on peut mettre plus d'icône dans la barre en bas svp

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Jan 21, 2025

@Nicolasfr340 @Quentin_17 vous en pensez quoi de la suggestion de @codeuriii ? 👍 Si vous êtes d'accord, 👎sinon

Perso, si vous êtes d'accord, je limiterai à 7 onglets pour les tablettes et ça reste à 5 pour les téléphones

@Clmnnt
Copy link
Contributor

Clmnnt commented Jan 21, 2025

@Nicolasfr340 @Quentin_17 vous en pensez quoi de la suggestion de @codeuriii ? 👍 Si vous êtes d'accord, 👎sinon

Perso, si vous êtes d'accord, je limiterai à 7 onglets pour les tablettes et ça reste à 5 pour les téléphones

1 slot en plus mais pas plus, ça fait beaucoup trop surchargé après

@Clmnnt
Copy link
Contributor

Clmnnt commented Jan 21, 2025

(enfin pour moi, a voir)

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Jan 25, 2025

J'attends une review de l'équipe Papillon pour voir si j'intègre ça

@codeuriii
Copy link
Contributor

On pourrait le rendre responsive genre plus la tablette est grande, plus on peut mettre d'onglets

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Feb 17, 2025

J'ai appliqué la suggestion de @Gabriel29306 pour détecter le mode tablette : tout simplement comparer le width et height
Ma méthode ne fonctionnait pas quand on mettait l'appli en fenêtre flottante

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request 🎨 interface Element visible to the user 🚸 user experience UX related issues
Projects
None yet
8 participants