Media Overlay component

Thanks for the component.
I am using the master branch of discourse github (commit https://github.com/discourse/discourse/commit/db7be947df29bb2ab27dd3abd80e3b470dcb74e5).
Just wanted to let you know that I’ve just enabled the component on the Default Theme.
Once I’ve done that, it breaks the rendering of the site, as it loops the main container again and again infinitely… any suggestion?
thanks!

1 me gusta

Thanks for the report @nicopace. @pmusaraj just pushed some updates to the theme

https://github.com/davidtaylorhq/discourse-media-overlay-theme/commit/b670c3125bf23859dd22ba68937a7b4d46f74eef

Please can you try updating and let us know if things are working better?

2 Me gusta

Thanks for the update. It works.
I wonder if there is any plan to support delayed iframe renderings like the ones in https://github.com/discourse/discourse-bbb that appear after you click in a button.

1 me gusta

Este (bastante maravilloso) Componente Temático parece funcionar, pero está etiquetado como “roto”.

No puedo hacer que no funcione.

¿Podrían otros probar también y proporcionar resultados?

Si no está roto, ¿podemos quitar la etiqueta de “roto”?

Hola @denvergeeks. Parece que esto se marcó como roto debido a algunos problemas con las onebox de YouTube. ¿Puede confirmar si los videos de YouTube funcionan correctamente en su sitio con este componente temático?

Gracias @david – Tienes razón, está lanzando errores y fallando también con videos de YouTube y iframes, pero hice un fork y eliminé LazyYT en el .js y funciona después de eso…

https://github.com/davidtaylorhq/discourse-media-overlay-theme/commit/6fbc1b46b806d0fb4d2e7bc538e1e2ba0ef1

2 Me gusta

¡Genial! ¿Te importaría hacer un PR con ese cambio?

1 me gusta

OK, creo que hice bien ese PR (lo suficiente).

3 Me gusta

Se ve genial, gracias. Ahora he eliminado broken de este tema :rocket:

3 Me gusta

¡ok, esto es genial! gracias por arreglar @denvergeeks y @david :slight_smile:

2 Me gusta

No todos los usuarios incrustarán el código iframe, todos incrustarán directamente la URL del video. Quizás valga la pena implementar una herramienta para convertir la URL del video de YouTube en un código de inserción iframe

Estoy bastante seguro de que se puede crear un componente temático para esto y colocar un botón en la barra de herramientas del compositor. O este podría ser bifurcado con esa parte añadida.

Un botón adicional complicará la interfaz. Es necesario que la conversión de direcciones URL se realice de forma automática, imperceptible para el usuario. Entonces todos los videos incrustados, independientemente del método de incrustación, funcionarán correctamente.

1 me gusta

bueno ok. entonces se podría bifurcar y hacerlo automático, supongo. :woman_shrugging:t2:

2 Me gusta

¿Quizás debería estar en el núcleo para que todas las URL se analicen según el mismo estándar?

No estoy seguro de entender la solicitud aquí… Discourse ya expande los enlaces de YouTube a iframes como parte de nuestro sistema “onebox”. Simplemente pega la URL en una línea separada:

2 Me gusta

Sí, pero pegar la URL directamente no funciona con este componente. El componente no ve el iframe onebox. Necesitas pegar el código iframe directamente para que el componente funcione. Y lo que nadie hace, todos ponen solo la URL, no el código iframe.

2 Me gusta

Sí, lo ideal sería que los usuarios simplemente pegaran lo que se proporciona en la fuente y ¡funcionara automáticamente en Discourse!

¡Parece bastante básico para los novatos!

1 me gusta

Ah, ya veo, tiene sentido :+1:

Definitivamente pr-welcome si alguien quiere intentar que este componente temático funcione con la nueva implementación de ‘videos perezosos’ del núcleo.

4 Me gusta

Gracias de nuevo. El componente temático Media Overlay de GitHub - davidtaylorhq/discourse-media-overlay-theme fue la razón. Cuando lo desactivé, Discourse comenzó a funcionar correctamente.

1 me gusta