I tweet con citazione di Twitter non vengono mostrati nelle onebox

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 Mi Piace

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 Mi Piace

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

9 Mi Piace

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

13 Mi Piace

La PR è in forno qui:

@techAPJ la sta revisionando.

Ci sono alcune piccole cose CSS che @awesomerobot potrebbe voler guardare. In generale non ci piace usare :hover e gli angoli arrotondati lì sembrano più grandi di quelli di Twitter, ma dovrebbe essere banale da correggere.

Nel complesso sta venendo benissimo @jcalvento!

7 Mi Piace

La PR sembra fantastica, @jcalvento! :+1:

Ho aggiunto un piccolo commento riguardo alla formattazione del codice; una volta completato, sarà pronta per il merge. :slight_smile:

4 Mi Piace

Questo cambiamento è ora attivo :tada:

Grazie @jcalvento :trophy:

10 Mi Piace

Lo spazio sembra eccessivo su mobile, può essere ridotto?

6 Mi Piace

Ho apportato alcune modifiche per migliorare lo stile generale delle citazioni.

A mio avviso, c’è ancora un margine minimo per ridurre lo spazio tra:

  • nome/username e corpo del tweet
  • tweet citato e data/ora/like/retweet

Lo spazio deriva da white-space: pre-line applicato alla classe tweet. @awesomerobot, puoi dare un’occhiata veloce per vedere se c’è spazio per ulteriori miglioramenti?

11 Mi Piace

Ancora troppo spazio su mobile.

1 Mi Piace

Esatto, ne abbiamo bisogno affinché gli spazi bianchi nel tweet vengano formattati come sono stati pubblicati… ma fa anche sì che gli spazi bianchi nell’HTML vengano visualizzati, quindi qui l’ho rimosso nel template: questo dovrebbe risolvere il problema

5 Mi Piace

Ho distribuito il gem aggiornato e la onebox sembra molto meglio!

Dovremmo considerare di ridurre il padding nella parte superiore del timestamp a 5px (da 10px)?

8 Mi Piace

Oh, penso di aver dimenticato degli spazi vuoti in fondo… Penso che questo lo risolverà

5 Mi Piace

Non lo so, penso che Arpit abbia ragione quando dice

2 Mi Piace

Ancora troppo spazio bianco qui @awesomerobot

3 Mi Piace

Hmm, c’è ancora un ritorno a capo in eccesso nel template che mi sono perso da qualche parte… ma rimuovendo il padding si eliminerà comunque lo spazio.

5 Mi Piace