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 ![]()
PR が準備完了です:
@techAPJ がレビュー中です。
@awesomerobot にも見てほしい小さな CSS の修正点がいくつかあります。一般的に :hover の使用は好ましくなく、そこにある角丸も Twitter のものより大きく見えますが、修正は簡単です。
全体として素晴らしい出来ですね @jcalvento !
PR、素晴らしいですね @jcalvento! ![]()
コードのフォーマットに関する小さなコメントを追加しました。それが完了すれば、マージの準備が整います。![]()
この変更は現在ライブです ![]()
@jcalvento ありがとう ![]()
全体の引用スタイルを改善するために、いくつかの変更 を行いました。
私の考えでは、以下の間のスペースをさらに減らす余地がまだ少しあります。
このスペースは、tweet クラスに適用されている white-space: pre-line に由来しています。@awesomerobot さん、さらなる改善の余地があるか、少し確認していただけませんか?
モバイルではまだ余白がとにかく多すぎます。
その通りです。ツイート内の空白を投稿時のまま表示するためには必要ですが、その結果 HTML 内の空白も表示されてしまいます。そこでテンプレートからこれを削除しました。これで解決するはずです。
アップデートされた gem をデプロイし、onebox の見た目が大幅に改善されました!
タイムスタンプの上のパディングを 10px から 5px に減らすことを検討すべきでしょうか?
あ、下の方に空白をいくつか見落としていたかもしれません…これで修正できると思います
わかりませんが、アルピットが言っていることは正しいと思います。
ここもまだ余白が多すぎます @awesomerobot
うーん、テンプレート内でまだ見落としている余計な改行がどこかにあるみたいですが… パディングを削除すれば、それでもスペースはなくなります。