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 ![]()
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!
La PR sembra fantastica, @jcalvento! ![]()
Ho aggiunto un piccolo commento riguardo alla formattazione del codice; una volta completato, sarà pronta per il merge. ![]()
Questo cambiamento è ora attivo ![]()
Grazie @jcalvento ![]()
Ho apportato alcune modifiche per migliorare lo stile generale delle citazioni.
A mio avviso, c’è ancora un margine minimo per ridurre lo spazio tra:
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?
Ancora troppo spazio su mobile.
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
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)?
Oh, penso di aver dimenticato degli spazi vuoti in fondo… Penso che questo lo risolverà
Non lo so, penso che Arpit abbia ragione quando dice
Ancora troppo spazio bianco qui @awesomerobot
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.