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

Feature/mobile marigolds #441

Merged
merged 8 commits into from
Jan 24, 2017
Merged

Conversation

Simounet
Copy link
Member

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 ?

@Simounet Simounet added this to the Version 1.8 milestone Jan 21, 2017
@Simounet Simounet requested a review from Sbgodin January 21, 2017 21:30
@Sbgodin Sbgodin self-assigned this Jan 21, 2017
@Sbgodin
Copy link
Member

Sbgodin commented Jan 21, 2017

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 !

@Simounet
Copy link
Member Author

Pourrais-tu me faire une capture d'écran pour être bien sûr de ce qui te dérange stp ?

@Sbgodin
Copy link
Member

Sbgodin commented Jan 22, 2017

leed lu decale

Avant, aucun titre n'était sur deux lignes, le texte excédentaire était tronqué par des "...". Les Lu étaient bien alignés, calés sur la droite. Chaque ligne avait son Lu et chaque Lu avait sa ligne.

@Simounet
Copy link
Member Author

C'était vraiment très moche. Désolé. J'ai mis tout ça à jour.
J'en ai profité pour mettre à jour fleaditlater en supprimant le style inline et en réutilisant le code poussé dans marigolds donc si tu peux récupérer ça, ce serait cool.

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.

@Sbgodin
Copy link
Member

Sbgodin commented Jan 22, 2017

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.

@Simounet
Copy link
Member Author

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.

@Sbgodin
Copy link
Member

Sbgodin commented Jan 22, 2017

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.

@Simounet
Copy link
Member Author

Vraiment étrange. Es-tu sûr d'avoir mis Leed à jour par rapport à cette branche ? Je n'arrive pas à le reproduire.
screen shot 2017-01-22 at 17 48 58

@Sbgodin
Copy link
Member

Sbgodin commented Jan 22, 2017

Leed est à da23b98 et FleedItLater est à b2232a9. En plus, je me suis trompé : le comportement se répercute sur l'affichage à droite des dossiers de flux.

@Simounet
Copy link
Member Author

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. :/

@Simounet
Copy link
Member Author

Je viens de pousser une gestion plus fine. Pourrais-tu tester ça @Sbgodin s'il te plaît ?

@Sbgodin
Copy link
Member

Sbgodin commented Jan 23, 2017

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 :

  1. Le commit actuel (674fafd: Marigolds word...) produit un affichage correct.
  2. Le commit précédent (da23b98: Marigolds: suppression style...) produit un affichage correct.

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.

@Sbgodin
Copy link
Member

Sbgodin commented Jan 23, 2017

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.
Ou bien je peux le faire moi-même ^^

@Simounet
Copy link
Member Author

Hello,
Pas de soucis, j'ai fait un rebase de mon côté, c'était pas bien compliqué et c'est plus propre. Pourras-tu tester le résultat de ton côté ?
On va y arriver !

@Sbgodin
Copy link
Member

Sbgodin commented Jan 24, 2017

J'ai testé. Aucun soucis. Merci ^^

@Simounet
Copy link
Member Author

Cool ! Je peux donc merger ?

@Sbgodin
Copy link
Member

Sbgodin commented Jan 24, 2017

Tout-à-fait ! Merci beaucoup pour ces améliorations.

@Simounet Simounet merged commit 88d8447 into LeedRSS:dev Jan 24, 2017
@Simounet
Copy link
Member Author

Avec plaisir. 😄

@Simounet Simounet deleted the feature/mobile-marigolds branch January 24, 2017 12:37
@Sbgodin
Copy link
Member

Sbgodin commented Jan 25, 2017

J'ai trouvé pourquoi l'affichage était décalé : j'ai fixé une taille minimale de caractère à 12.

@Simounet
Copy link
Member Author

Mais est-ce que tu as toujours le soucis ?

@Sbgodin
Copy link
Member

Sbgodin commented Jan 25, 2017

Une fois retirée cette taille minimale, Leed s'affichait correctement.

@Simounet
Copy link
Member Author

Ok mais si tu remets la taille qui posait problème avec le code d'aujourd'hui, il n'y a plus de soucis ?

@Sbgodin
Copy link
Member

Sbgodin commented Jan 26, 2017

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.

@Simounet
Copy link
Member Author

J'ai un peu de mal à comprendre car je viens de tester en me mettant sur master et je constate bien le décalage des compteurs sur les flux lorsque je passe la typo en dessous de 16px. Dans tous les cas, marigolds n'est pas vraiment fluide. Peut-être un jour…

@Sbgodin
Copy link
Member

Sbgodin commented Jan 26, 2017

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants