Twitterの引用ツイートがワンボックスに表示されない

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

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

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

「いいね!」 9

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

「いいね!」 13

PR が準備完了です:

@techAPJ がレビュー中です。

@awesomerobot にも見てほしい小さな CSS の修正点がいくつかあります。一般的に :hover の使用は好ましくなく、そこにある角丸も Twitter のものより大きく見えますが、修正は簡単です。

全体として素晴らしい出来ですね @jcalvento

「いいね!」 7

PR、素晴らしいですね @jcalvento! :+1:

コードのフォーマットに関する小さなコメントを追加しました。それが完了すれば、マージの準備が整います。:slight_smile:

「いいね!」 4

この変更は現在ライブです :tada:

@jcalvento ありがとう :trophy:

「いいね!」 10

モバイル表示では間隔が極端に見えます。詰めることはできますか?

「いいね!」 6

全体の引用スタイルを改善するために、いくつかの変更 を行いました

私の考えでは、以下の間のスペースをさらに減らす余地がまだ少しあります。

  • 名前/ユーザー名とツイート本文の間
  • 引用ツイートとタイムスタンプ/いいね/リツイートの間

このスペースは、tweet クラスに適用されている white-space: pre-line に由来しています。@awesomerobot さん、さらなる改善の余地があるか、少し確認していただけませんか?

「いいね!」 11

モバイルではまだ余白がとにかく多すぎます。

「いいね!」 1

その通りです。ツイート内の空白を投稿時のまま表示するためには必要ですが、その結果 HTML 内の空白も表示されてしまいます。そこでテンプレートからこれを削除しました。これで解決するはずです。

「いいね!」 5

アップデートされた gem をデプロイし、onebox の見た目が大幅に改善されました!

タイムスタンプの上のパディングを 10px から 5px に減らすことを検討すべきでしょうか?

「いいね!」 8

あ、下の方に空白をいくつか見落としていたかもしれません…これで修正できると思います

「いいね!」 5

わかりませんが、アルピットが言っていることは正しいと思います。

「いいね!」 2

ここもまだ余白が多すぎます @awesomerobot

「いいね!」 3

うーん、テンプレート内でまだ見落としている余計な改行がどこかにあるみたいですが… パディングを削除すれば、それでもスペースはなくなります。

「いいね!」 5