-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
fix(home) : Utilisation plus simple de la page d'accueil (Android) + Quelques nouveautés #372
Conversation
This reverts commit 5bcae66.
…ariables non utilisées
Comment ? |
non ?? |
Ben la majorité des personnes se plaignent d'une page d'accueil instable depuis la version 7.3 |
précision : sur android |
Je sais ! Elle pas forcément instable, j'ai pu tester sur 2 téléphone bas de gamme, ça fonctionne ! C'est juste effectivement moyennement stable ! MAIS c'est juste la tick speed qui est trop haute sur android ! |
Donc faut changer la tick speed ou faire un revert et reprendre après ? @godetremy |
On fix ! Je préfère que vous corriger plutôt que retirer ! |
ça dépend, elle était terminé au moment du merge ? autrement vaut mieux que tu la reprennes de ton côté @godetremy |
je vais voir ce que je peux faire, mais je garantis rien |
Sinon, il est possible de garder les composants avec |
OK je me le note pour quand j'ai le temps 👍 |
Comme ça ? |
pas mieux |
la c'est bon merci @Yaya-Cout |
Du coup, penses tu @LeGeek01 qu'il faut conserver cette nouvelle page d'accueil ? |
Bah au vu des bugs que je vois, visiblement elle semble assez potable sur ios même si y'a le bug du carré que ora a envoyé, sur android c'est affreux, je suis d'avis de ne pas la garder en production pour le moment jusqu'à ce que les bugs soit réglés |
Ou du moins de laisser comme dans cette pr, la nouvelle page sur ios et garder l'ancienne pour Android |
Oui je suis d'accord |
A la demande, je mets cette pr en draft pour faire un fix de la nouvelle page d'accueil plutôt que de faire un revert |
@godetremy comme tu me l'as demandé, la page d'accueil a été corrigé sur Android, en fonction du défilement, l'arrière plan s'adapte et pour éviter une page d'accueil vide (notamment le week-end), des messages sont affichées (voir captures dans la description de la pr) J'espère que la pr sera merged vite :) |
bon, ça fait 1 mois, ma patience a des limites |
🚀 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].
TODO
(aka des annotations pour du code manquant) dans vos modificationsChangelogs proposés
Capture(s) d'écran
Issues en lien
Informations supplémentaires
Provenant de #325
Fait en partie par @Gabriel29306