Twitter-Zitate werden in Oneboxes nicht angezeigt

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 „Gefällt mir“

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 „Gefällt mir“

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

9 „Gefällt mir“

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

13 „Gefällt mir“

Der PR ist hier fast fertig:

@techAPJ überprüft ihn gerade.

Es gibt ein paar kleinere CSS-Angelegenheiten, die @awesomerobot sich ebenfalls ansehen könnte. Im Allgemeinen mögen wir :hover nicht so sehr, und die abgerundeten Ecken dort wirken etwas größer als die bei Twitter, aber das sollte leicht zu beheben sein.

Insgesamt sieht das super aus, @jcalvento!

7 „Gefällt mir“

Der PR sieht super aus, @jcalvento! :+1:

Ich habe einen kleinen Kommentar zur Code-Formatierung hinzugefügt. Sobald das erledigt ist, kann er gemergt werden. :slight_smile:

4 „Gefällt mir“

Diese Änderung ist jetzt live :tada:

Danke an @jcalvento :trophy:

10 „Gefällt mir“

Der Abstand auf Mobilgeräten wirkt extrem. Kann er verringert werden?

6 „Gefällt mir“

Ich habe einige Änderungen vorgenommen, um das allgemeine Zitat-Design zu verbessern.

Meiner Meinung nach gibt es noch ein wenig Spielraum, um den Abstand zwischen

  • Name/Benutzername und Tweet-Text
  • zitiertem Tweet und Zeitstempel/Likes/Retweets

zu verringern.

Der Abstand ergibt sich aus white-space: pre-line, das auf die Klasse tweet angewendet wird. @awesomerobot, könntest du kurz prüfen, ob es noch weitere Verbesserungsmöglichkeiten gibt?

11 „Gefällt mir“

Immer noch viel zu viel Platz auf dem Handy.

1 „Gefällt mir“

Richtig, das brauchen wir, damit der Leerzeichenabstand im Tweet so formatiert wird, wie er gepostet wurde. Allerdings sorgt es auch dafür, dass Leerzeichen im HTML angezeigt werden. Deshalb habe ich es hier in der Vorlage entfernt – das sollte das Problem beheben.

5 „Gefällt mir“

Das aktualisierte Gem wurde deployed, und die Onebox sieht jetzt viel besser aus!

Sollten wir den Abstand oben beim Zeitstempel von 10px auf 5px verringern?

8 „Gefällt mir“

Oh, ich glaube, ich habe unten etwas Leerzeichen vergessen… Ich denke, das wird es beheben.

5 „Gefällt mir“

Keine Ahnung, ich finde, Arpit hat recht, wenn er sagt:

2 „Gefällt mir“

Immer noch zu viel Weißraum hier @awesomerobot

3 „Gefällt mir“

Hmm, irgendwo im Template ist noch eine zusätzliche Zeilenumbruch, die ich übersehen habe… aber wenn man den Padding entfernt, verschwindet der Abstand trotzdem.

5 „Gefällt mir“