-
Notifications
You must be signed in to change notification settings - Fork 41
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
Feature/mobile marigolds #441
Conversation
Les boutons sont plus gros et accessibles. +1. Par contre, les boutons Lu sont disposés en diagonale avec le greffon FleadItLater, comme des slashs (/). Pas de soucis quand même, j'imagine que cela aura des effets de bord sur d'autres composants. C'est bon pour moi, merci pour tout ça ! |
Pourrais-tu me faire une capture d'écran pour être bien sûr de ce qui te dérange stp ? |
C'était vraiment très moche. Désolé. J'ai mis tout ça à jour. Autre petite mise à jour, la gestion du truncate qui ne prenait pas en compte les caractères spéciaux. Maintenant, quand on lui dit de couper à 37, il ne coupe plus à 27 vrais caractères. |
Sur mobile, impec. Par contre, sur PC c'est devenu comme je l'ai décrit : les Lu décalés. J'ai mis à jour FleedItLater. |
C'est juste au niveau des fleaditlater ou des autres flux également ? Les flux sont trop longs ? Faudrait vraiment que je supprime ces float de partout mais je t'avoue avoir un peu la flemme, surtout que je n'utilise Marigolds que pour les settings au quotidien. |
Tout se passe uniquement au niveau de fleadItLater. Et c'est exactement comme sur la capture d'écran du téléphone mobile, sauf la taille. Actuellement, c'est bien affiché sur le téléphone, pas bien affiché sur l'ordinateur. Sur ordinateur, en réduisant suffisamment la largeur, l'affichage devient équivalent à celui sur mobile. C'est-à-dire correct, actuellement. |
Hmmm, désolé d'insister mais n'aurais-tu pas du cache front ? J'avais bien ce comportement avant mes dernières modifs mais tout est ok depuis. :/ |
Je viens de pousser une gestion plus fine. Pourrais-tu tester ça @Sbgodin s'il te plaît ? |
Mmh... En fait, c'est de ma faute. Je ne sais pas encore pourquoi. Mon navigateur habituel, endurci contre la pub, maltraite Leed pourtant épargné par les filtres. Par contre, mon navigateur de développement ainsi que mon téléphone indiquent que :
Bref, j'ai eu tord de ne pas tester dans mon environnement de développement. Au temps pour moi :-/ Alors tu peux fusionner sans soucis. |
J'ai fait une requête de fusion sur ta requête de fusion ^^ Vu qu'il y a un conflit à cause de la fusion que j'ai faite juste avant... Ma requête corrige donc le conflit introduit par la fusion d'avant. Bon... Donc... Tu incorpores ma fusion et alors la fusion d'ici ne fait plus de conflit avec la branche fusionnée avant. Normalement. EDIT: bon, ça ne marche pas mieux. Le conflit est là quand même. Tout ce qu'il faut faire pour résoudre le conflit est de remplacer ONE_TIME_PASSWORD par OTP_CODE. |
674fafd
to
b60d048
Compare
Hello, |
J'ai testé. Aucun soucis. Merci ^^ |
Cool ! Je peux donc merger ? |
Tout-à-fait ! Merci beaucoup pour ces améliorations. |
Avec plaisir. 😄 |
J'ai trouvé pourquoi l'affichage était décalé : j'ai fixé une taille minimale de caractère à 12. |
Mais est-ce que tu as toujours le soucis ? |
Une fois retirée cette taille minimale, Leed s'affichait correctement. |
Ok mais si tu remets la taille qui posait problème avec le code d'aujourd'hui, il n'y a plus de soucis ? |
Dès que je met une taille minimale de police à 12, le décalage est de retour dans certaines révisions. dès que j'enlève la taille minimale, aucune révision ne présente de décalage. Avant b7971a3 le décalage n'apparaît pas, mais apparaît avec cette révision. À partir de b7971a3, les boutons du menu de droite sont décalés. Pas ceux de FleadItLater. À partir de 6228b16, les boutons du menu de droite et ceux de FleadItLater sont décalés. |
J'ai un peu de mal à comprendre car je viens de tester en me mettant sur |
Chez moi master (58cfbb1) produit un affichage correct... Cela doit certainement dépendre du contenu des titres de flux et de l'âge du capitaine. |
J'ai fait un petit coup de peinture dans l'admin. Ce n'est pas encore parfait mais c'est mieux que ce qu'il y avait. Un petit test/avis stp ?