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 curtida

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 curtidas

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 curtida

Este componente de tema (bastante maravilhoso) parece funcionar, mas está marcado como “quebrado”.

Não consigo fazer com que ele não funcione.

Outros poderiam testar também e fornecer resultados?

Se não estiver quebrado, podemos remover a tag “quebrado”?

Olá @denvergeeks. Parece que isso foi marcado como quebrado devido a alguns problemas com as oneboxes do YouTube. Você pode confirmar se os vídeos do YouTube estão funcionando bem em seu site com este componente de tema?

Obrigado @david – Você está correto, está gerando erros e falhando em vídeos do YouTube e iframes também, mas fiz um fork e removi o LazyYT no .js e funciona depois disso…

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

2 curtidas

Ótimo! Você se importaria de fazer um PR com essa alteração?

1 curtida

OK, acho que fiz esse PR certo (o suficiente).

3 curtidas

Parece ótimo, obrigado por isso. Removi o broken deste tópico :rocket:

3 curtidas

ok, isso é incrível! obrigado por consertar @denvergeeks e @david :slight_smile:

2 curtidas

Nem todo usuário incorporará o código iframe, todos incorporarão diretamente o URL do vídeo. Talvez valha a pena implementar uma ferramenta para converter o URL do vídeo do YouTube em um código de incorporação iframe

Tenho certeza de que um componente de tema pode ser feito para isso e colocar um botão na barra de ferramentas do compositor. Ou este poderia ser bifurcado com essa parte adicionada.

Um botão adicional complicará a interface. É necessário que a conversão de endereços URL seja realizada automaticamente, imperceptivelmente para o usuário. Assim, todos os vídeos incorporados, independentemente do método de incorporação, funcionarão corretamente.

1 curtida

bem ok. então poderíamos fazer um fork e torná-lo automático, suponho. :mulher_dando_de_ombros:t2:

2 curtidas

Talvez devesse estar no núcleo para que todos os URLs sejam analisados com o mesmo padrão?

Não tenho certeza se entendi a solicitação aqui… O Discourse já expande links do YouTube em iframes como parte do nosso sistema "onebox". Basta colar o URL em uma linha separada:

2 curtidas

Sim, mas colar a URL diretamente não funciona com este componente. O componente não vê o onebox do iframe. Você precisa colar o código do iframe diretamente para que o componente funcione. E o que ninguém faz, todos colocam apenas a URL, não o código do iframe.

2 curtidas

Sim, o ideal seria que os usuários simplesmente colassem o que é fornecido pela fonte e funcionasse automaticamente no Discourse!

Parece bem básico para novatos!

1 curtida

Ah, entendi, faz sentido :+1:

Definitivamente pr-welcome se alguém quiser tentar fazer este componente de tema funcionar com a nova implementação de ‘lazy-videos’ do core.

4 curtidas

Obrigado novamente. O componente de tema Media Overlay de GitHub - davidtaylorhq/discourse-media-overlay-theme foi o motivo. Quando o desativei, o Discourse começou a funcionar corretamente.

1 curtida