Les tweets cités de Twitter ne s'affichent pas dans les oneboxes

Twitter oneboxes work well, except for quote tweets. In this case, the “parent” tweet is shown, but the quoted tweet is only shown as a link.

Would appreciate if someone could improve the oneboxing of quote tweets.

Quote tweet as displayed on Twitter:

Onebox:

11 « J'aime »

I like this suggestion, its a rather complex change, but wacking a pr-welcome on it for now in case anyone is feeling brave.

5 « J'aime »

I’ll try to make a PR to fix this. Will let you know

9 « J'aime »

Still working on it but let me know if you have any suggestions/comments :slight_smile:

13 « J'aime »

La PR est en cours de préparation ici :

@techAPJ est en train de la réviser.

Il y a quelques petits détails CSS que @awesomerobot pourrait également examiner. En général, nous n’aimons pas trop utiliser :hover, et les coins arrondis semblent plus grands que ceux de Twitter, mais cela devrait être facile à corriger.

Dans l’ensemble, ça a l’air super bien, @jcalvento !

7 « J'aime »

La PR a l’air super, @jcalvento ! :+1:

J’ai ajouté un petit commentaire concernant la mise en forme du code. Une fois cela fait, elle sera prête à être fusionnée. :slight_smile:

4 « J'aime »

Ce changement est maintenant en ligne :tada:

Merci @jcalvento :trophy:

10 « J'aime »

L’espacement semble excessif sur mobile, peut-on le réduire ?

6 « J'aime »

J’ai apporté quelques modifications pour améliorer le style général des citations.

À mon avis, il reste encore un peu de marge pour réduire l’espace entre :

  • le nom/pseudonyme et le corps du tweet
  • le tweet cité et l’horodatage/les likes/les retweets

L’espace provient de white-space: pre-line appliqué à la classe tweet. @awesomerobot, peux-tu jeter un coup d’œil rapide pour voir s’il y a une possibilité d’amélioration supplémentaire ?

11 « J'aime »

Toujours beaucoup trop d’espace sur mobile.

1 « J'aime »

Exactement, nous en avons besoin pour que les espaces dans le tweet soient formatés comme à l’envoi… mais cela fait aussi apparaître les espaces dans le HTML, alors je les ai supprimés ici dans le modèle — cela devrait régler le problème

5 « J'aime »

Le gem mis à jour a été déployé et la onebox a une bien meilleure apparence !

Devrions-nous envisager de réduire le padding en haut du timestamp à 5px (au lieu de 10px) ?

8 « J'aime »

Oh, je pense que j’ai oublié quelques espaces en bas… Je pense que cela devrait régler le problème.

5 « J'aime »

Je ne sais pas, je pense qu’Arpit a raison quand il dit

2 « J'aime »

Toujours trop d’espaces blancs ici @awesomerobot

3 « J'aime »

Hmm, il reste encore un retour à la ligne en trop dans le modèle que j’ai manqué quelque part… mais supprimer le padding éliminera l’espace malgré cela.

5 « J'aime »