-
Notifications
You must be signed in to change notification settings - Fork 26
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
✨ [Homework] : Ajout du sélecteur de date et d'un panneau de réglage de l'onglet - 🚨 iOS ONLY 🚨 #158
✨ [Homework] : Ajout du sélecteur de date et d'un panneau de réglage de l'onglet - 🚨 iOS ONLY 🚨 #158
Conversation
Si on veut garder le même style pour Android, on va devoir recoder un calendrier entier. Ou sinon adapter la modale. À voir |
Pourquoi changer le style du calendrier ? |
Car comme dit plutôt, Android ne propose pas de style inline pour les calendriers. |
Je suis en train de faire un calendrier qui respecte le design de Papillon, je l’envoie cette après-midi ! |
De plus, ce calendrier a été codé dans le mauvais fichier |
J'aime bien, à quoi ressemble le menu défilant pour Android ? |
Bonne chance hein 🤷 |
@Gabriel29306 je crois qu'on tien un truc là ? |
Il y a juste l'énorme menu défilant qui est moche, à cause d'Android, sinon c'est bon. |
Je vais essayer d'intégrer la roue sur Android |
Je sais pas si c'est possible |
Moyen convaincu par vos propositions, autant faire un truc propre sur iOS, puis attendre d'avoir un truc propre sur Android pour pouvoir l'intégrer... |
Faut encore que tes image soit visible |
Niveau visuel, j'aime bien ce que Remy propose, bravo ! |
Quel version aimes-tu ? Celle de @godetremy ou la mienne ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testé de mon côté de façon très basique, pas encore d'analyse détaillée du code ni des possibles erreurs.
J'ai edit mon commentaire sans voir ton message, sry |
Je n'ai rien posté sur ce PR mais sur mon fork |
Y'a volonté de nuire !!! |
Pas de chance maintenant on travail à deux sur une version Android mais bref... |
Au pire @godetremy, tu ne veux pas faire en sorte que quand l'utilisateur choisi le mercredi de la semaine, ça le scroll automatique sur le mercredi de cette dite semaine, comme ça, c'est logique selon lui. ;) |
Je vois pas le rapport, il a bien qualifié de "moche" la version sur Android, c'est un fait. Il n'a pas dit "kiffer" le design |
Mais connard tu comprends pas quoi dans le concept de demander à l'utilisateur l'intervalle de dates qui correspond à la semaine? c'est de la logique ! |
OUI.... |
Bah écoute, je crois qu'on a vraiment pas la même logique mdr |
Mais de toute façon ça parait simple : tant que les deux plateformes n'ont pas la même UX la PR ne sera pas acceptée. Papillon est multiplateforme mais elle reprend les mêmes logiques pour chaque OS. Ici le fait d'avoir une liste sur android ne correspond tout simplement pas avec le choix fait sur iOS (qui oui est meilleur). |
Donc on ai d'accord que la wheel est une bonne idée ? |
Oui mais il attend mon interprétation sur android hors la 'est uniquement un proto, je n'ai rien PR |
Mais remet le en draft, merde |
Bah nan, la PR est ready, il a mal compris dans son message, il a dit que la PR voulait mettre une wheel sur iOS et un select sur Android... hors là c'est ton idée et pas celle de la PR |
Bah oui mais il attend un truc de prod pour les 2 OS |
mb j'avais mal capté le tournant de la PR et oui c'est bien le calendrier classique qu'il faudrait refaire à l'identique |
Bon bah faut vraiment aller se faire voir ... |
C'est vraiment pas logique votre histoire mais bref... |
Puedo faire le chiant ?
Celle de Tom :
Par contre, je trouve qu’il faudrait mettre le bouton pour les devoirs faits ailleurs, car ça veut dire qu’il faudrait obligatoirement passer par le calendrier/la roue pour masquer les devoirs fait. |
Et en tant qu’utilisateur normal, qui va juste défilé la semaine d’après ou juste quelques semaines après, et qui n’est pas (encore) développeur, bah je préfère la version de Tom, mais c’est à vous de voir. Car je trouve que c’est plus rapide/compréhensible pour l’utilisateur lambda, qui ne va pas lire les détails écrit au dessus des calendriers + pas savoir c’est qu’elle pour qu’elle jour son devoir voir même qu’elle jour il est (même si le jour en question est coloré). Alors qu’à contrario un mec normal si il sait que c’est 2 semaines après, il ouvre la roue, descend 2 fois et hop c’est good, sans même savoir c’est quelle jour et quelle jour il est actuellement |
Bon c'est pas compliqué, on va créer le composant PapillonCalendar ça serra plus simple visiblement... |
Je vois ce que tu veux dire, mais sa version comprend plusieurs désavantages :
Et par rapport à la simplicité de faire +2 semaines, on peut dire de même pour celle de Rémy, elle offre même de la précision selon moi. |
Le calendrier est quand-même beaucoup plus lisible que la roue, car tu as le mois visible facilement (c'est moins facile à lire sous forme de nombre), et que quand tu sais que tu as un travail à faire pour le 9 septembre, tu n'as pas à réfléchir à quelle semaine c'est en regardant les intervalles mais juste à cliquer sur le bon jour. |
Le problème c'est comment ajouter les paramètre de l'onglet sur android ? |
@godetremy ne respecte pas le design Papillon à moins que c'est un changement surprise ? |
bah après pour moi c'est une question d'UX localisée, à titre perso j'ai l'habitude de voir un calendrier assez lambda et ça m'a jamais dérangé plus que ça, à titre purement personel je m'axe plutôt sur la proposition de rémy par contre tom je te trouve assez coincé du postérieur et ça me plaît pas |
Pronote utilise un calendrier, d'ailleurs (sur téléphone en tout cas). |
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
Ajout du sélecteur de date et d'un panneau de réglage de l'onglet
Informations supplémentaires
🚨 iOS ONLY 🚨
Une pull request est à prévoir pour Android ! Cela est lié au fait qu'Android ne propose pas de sélecteur de date en inline.