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.
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.
I like this suggestion, its a rather complex change, but wacking a pr-welcome on it for now in case anyone is feeling brave.
I’ll try to make a PR to fix this. Will let you know
Still working on it but let me know if you have any suggestions/comments ![]()
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!
Der PR sieht super aus, @jcalvento! ![]()
Ich habe einen kleinen Kommentar zur Code-Formatierung hinzugefügt. Sobald das erledigt ist, kann er gemergt werden. ![]()
Diese Änderung ist jetzt live ![]()
Danke an @jcalvento ![]()
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
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?
Immer noch viel zu viel Platz auf dem Handy.
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.
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?
Oh, ich glaube, ich habe unten etwas Leerzeichen vergessen… Ich denke, das wird es beheben.
Keine Ahnung, ich finde, Arpit hat recht, wenn er sagt:
Immer noch zu viel Weißraum hier @awesomerobot
Hmm, irgendwo im Template ist noch eine zusätzliche Zeilenumbruch, die ich übersehen habe… aber wenn man den Padding entfernt, verschwindet der Abstand trotzdem.