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.

Sign up for GitHub

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

Jump to bottom

fix(home): Utilisation correcte sur Android + Non vide le week-end (Page d'accueil, V2) #483

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

fix(home): Utilisation correcte sur Android + Non vide le week-end (Page d'accueil, V2) #483

wants to merge 54 commits into from
+401 −294

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Dec 16, 2024 •
edited
Loading

🚀 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

  • Utilisation plus facile de la page d'accueil
  • En fonction du défilement, l'arrière-plan s'adapte
  • Fix de l'affichage incorrect du nom du compte en mode clair
  • Affichage d'un message lorsqu'aucune absence/note/devoir…

Capture(s) d'écran

Avant Après
1733158800565 1733158800570

Issues en lien

Informations supplémentaires

Copier-coller de #372

Kgeek33 and others added 30 commits November 16, 2024 21:32
@Kgeek33
Revert "!feat(Home): Rewritten Home for performance issue"
6d69f24 
This reverts commit 5bcae66.
@Kgeek33
fix typage + intégration des sections par ordre d'importance
6cab818
@Kgeek33
fix affichage incorrct du nom du compte en mode clair + suppression v…
c27e8a7 
…ariables non utilisées
@Kgeek33
fix non affichage de l'influence depuis l'accueil
1e5494a
@Kgeek33
fix affichage vide quand salle inconnue
fb7e1c3
@Kgeek33
Merge branch 'main' into fix/home
ff9ba69
@Kgeek33
Merge branch 'main' into fix/home
290f602
@Kgeek33
Rename Home.tsx to Home.android.tsx
5f36733
@Kgeek33
Create Home.ios.tsx
dfa979e
@Kgeek33
Create ModalContent.tsx
488855f
@Kgeek33
Update Home.android.tsx
27ee23b
@Kgeek33
Update Home.ios.tsx
7c63aed
@Kgeek33
affichage corect des pages en fonction du système
16c07ed
@Kgeek33
fix import des accueils
f961e15
@Kgeek33
Merge branch 'main' into fix/home
93725c7
@Kgeek33
Merge branch 'main' into fix/home
21a4295
@Kgeek33
Merge branch 'main' into fix/home
8830d19
@Kgeek33
Merge branch 'main' into fix/home
329668b
@Kgeek33
affichage correct des devoirs le week-end + optimisations
9ec45b8
@Kgeek33
affichage d'un commentaire lorsque aucune absence
76cb1b3
@Kgeek33
affichage d'un commentaire lorsque aucune note
1bc296b
@Kgeek33
suppression des marginTop
4d205eb
@Kgeek33
nouvel émoji pour différencier des autres
c2a767c
@Kgeek33
Merge branch 'main' into fix/home
30480be
@Kgeek33
Merge branch 'main' into fix/home
9dda5a8
@Kgeek33
Merge branch 'main' into fix/home
ec90bfd
@Kgeek33
fix logs errors for style
3e892c8
@Kgeek33
suppression (comme en ce moment) de la page d'accueil sur Android
b18de09
@Kgeek33
delete PlaceholderScreen.tsx not use
f4e7707
@Kgeek33
delete code not use + indentation
d074eb9
@yannouuuu yannouuuu added the 🪱 bug Something isn't working label Dec 23, 2024
ecnivtwelve
ecnivtwelve requested changes Dec 25, 2024
View reviewed changes
Copy link
Contributor

@ecnivtwelve ecnivtwelve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai des influences très étranges sur iOS, cette PR devrait pas concerner uniquement Android ???

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2024-12-25.at.20.09.02.mp4

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 25, 2024

J'ai des influences très étranges sur iOS, cette PR devrait pas concerner uniquement Android ???

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2024-12-25.at.20.09.02.mp4

Oui ça devait que concerner que Android mais ça devait améliorer pour ios également
J'vais voir pour réaméliorer ça

@ecnivtwelve
Copy link
Contributor

J'ai des influences très étranges sur iOS, cette PR devrait pas concerner uniquement Android ???

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2024-12-25.at.20.09.02.mp4

Oui ça devait que concerner que Android mais ça devait améliorer pour ios également J'vais voir pour réaméliorer ça

en vrai te complique pas la tâche, applique tes fix android et on verra ensuite pour iOS (qui fonctionne bien à priori)

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 26, 2024

en vrai te complique pas la tâche, applique tes fix android et on verra ensuite pour iOS (qui fonctionne bien à priori)

ok oui, je vais faire ça et j'y travaillerai sur une autre pr (équité entre ios en android)

@Kgeek33 Kgeek33 requested a review from ecnivtwelve December 26, 2024 20:36
@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 26, 2024

Le rendu sur Android :

VID_20241226_215534.mp4

@ecnivtwelve
Copy link
Contributor

j'ai toujours de la dégression et des bugs sous iOS

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 27, 2024 •
edited
Loading

j'ai toujours de la dégression et des bugs sous iOS

c'est bizarre ça, je regarde qu'est-ce que j'ai oublié pour ios

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 27, 2024

@ecnivtwelve ok j'ai oublié les onscroll qu'il y avait sur ios, ça devrait être bon

@Clmnnt
Copy link
Contributor

Clmnnt commented Dec 27, 2024

@Kgeek33 oublie pas de sync pour le lint :))

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 27, 2024

@Kgeek33 oublie pas de sync pour le lint :))

ah là j'y peux rien, le lint a installé la dernière version de axios alors qu'on ne l'a pas installé (d'où ma pr #449 pour fix ça ;))

@Kgeek33 Kgeek33 mentioned this pull request Jan 3, 2025
Open
9 tasks
@Kgeek33 Kgeek33 mentioned this pull request Jan 6, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@godetremy godetremy Awaiting requested review from godetremy godetremy is a code owner

@ecnivtwelve ecnivtwelve Awaiting requested review from ecnivtwelve ecnivtwelve is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
🪱 bug Something isn't working
Projects
None yet
Milestone
No milestone
5 participants
@Kgeek33 @oriionn @ecnivtwelve @Clmnnt @yannouuuu