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 ![]()
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.
¡El PR se ve genial, @jcalvento! ![]()
Agregué un pequeño comentario sobre el formato del código; una vez que se complete, estará listo para fusionar. ![]()
Este cambio ya está activo ![]()
Gracias @jcalvento ![]()
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:
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?
Aún hay demasiadísimo espacio en móviles.
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.
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)?
Oh, creo que me faltó un poco de espacio en blanco al final… Creo que esto lo solucionará
No sé, creo que Arpit tiene razón cuando dice
Todavía hay demasiado espacio en blanco aquí @awesomerobot
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.