Los quote-tweets de Twitter no se muestran en oneboxes

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 Me gusta

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 Me gusta

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

9 Me gusta

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

13 Me gusta

El PR está en proceso aquí:

@techAPJ lo está revisando.

Hay algunos detalles menores de CSS que @awesomerobot podría querer revisar también. En general, no nos gusta usar :hover y las esquinas redondeadas allí parecen más grandes que las de Twitter, pero eso debería ser trivial de corregir.

En general, esto se ve genial, @jcalvento.

7 Me gusta

¡El PR se ve genial, @jcalvento! :+1:

Agregué un pequeño comentario sobre el formato del código; una vez que se complete, estará listo para fusionar. :slight_smile:

4 Me gusta

Este cambio ya está activo :tada:

Gracias @jcalvento :trophy:

10 Me gusta

El espaciado parece excesivo en móviles, ¿se puede reducir?

6 Me gusta

He realizado algunos cambios para mejorar el estilo general de las citas.

En mi opinión, todavía hay un pequeño margen para reducir el espacio entre:

  • nombre/usuario y el cuerpo del tweet
  • tweet citado y la marca de tiempo/me gusta/reenvíos

El espacio proviene de white-space: pre-line aplicado a la clase tweet. @awesomerobot, ¿podrías echar un vistazo rápido para ver si hay margen para mejorar aún más?

11 Me gusta

Aún hay demasiadísimo espacio en móviles.

1 me gusta

Correcto, lo necesitamos para que los espacios en blanco en el tuit se formateen tal como fueron publicados… pero también hace que los espacios en blanco en el HTML se muestren, así que aquí lo eliminé en la plantilla; esto debería solucionarlo.

5 Me gusta

Se ha implementado la gema actualizada y el onebox se ve mucho mejor.

¿Deberíamos considerar reducir el relleno superior del sello de tiempo a 5px (desde 10px)?

8 Me gusta

Oh, creo que me faltó un poco de espacio en blanco al final… Creo que esto lo solucionará

5 Me gusta

No sé, creo que Arpit tiene razón cuando dice

2 Me gusta

Todavía hay demasiado espacio en blanco aquí @awesomerobot

3 Me gusta

Hmm, aún hay un salto de línea extra en la plantilla que me perdí en algún lugar… pero eliminar el relleno eliminará el espacio a pesar de eso.

5 Me gusta