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 « J'aime »

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 « J'aime »

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 « J'aime »

Ce composant de thème (assez merveilleux) semble fonctionner, mais il est marqué comme « cassé ».

Je n’arrive pas à le faire ne pas fonctionner.

D’autres personnes pourraient-elles tester également et fournir des résultats ?

S’il n’est pas cassé, pouvons-nous supprimer la balise « cassé » ?

Salut @denvergeeks. Il semble que cela ait été marqué comme défectueux en raison de problèmes avec les oneboxes YouTube. Pouvez-vous confirmer si les vidéos YouTube fonctionnent correctement sur votre site avec ce composant de thème ?

Merci @david – Vous avez raison, cela génère des erreurs et bloque aussi les vidéos YouTube et les iframes, mais j’ai fait un fork et supprimé LazyYT dans le .js et cela fonctionne après cela…

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

2 « J'aime »

Super ! Pourriez-vous faire une PR avec cette modification ?

1 « J'aime »

OK, je pense que j’ai bien fait cette PR (assez).

3 « J'aime »

Ça a l’air super, merci. J’ai maintenant supprimé broken de ce sujet :rocket:

3 « J'aime »

ok c’est génial ! merci d’avoir corrigé @denvergeeks et @david :slight_smile:

2 « J'aime »

Tous les utilisateurs n’intégreront pas le code iframe, tout le monde intégrera directement l’URL de la vidéo. Il serait peut-être utile de mettre en œuvre un outil pour convertir l’URL de la vidéo YouTube en code d’intégration iframe

Je suis tout à fait certain qu’un composant de thème peut être créé pour cela et qu’un bouton peut être ajouté à la barre d’outils de l’éditeur. Ou celui-ci pourrait être “forké” avec cette partie ajoutée.

Un bouton supplémentaire compliquera l’interface. Il est nécessaire que la conversion des adresses URL s’effectue automatiquement, de manière imperceptible pour l’utilisateur. Ainsi, toutes les vidéos intégrées, quelle que soit la méthode d’intégration, fonctionneront correctement.

1 « J'aime »

Bien d’accord. On pourrait alors en faire un fork et le rendre automatique, je suppose. :woman_shrugging:t2:

2 « J'aime »

Peut-être devrait-il être dans le cœur pour que toutes les URL soient analysées selon la même norme ?

Je ne suis pas sûr de comprendre la demande ici… Discourse développe déjà les liens YouTube en iframes dans le cadre de notre système « onebox ». Il suffit de coller l’URL sur une ligne distincte :

2 « J'aime »

Oui, mais coller l’URL directement ne fonctionne pas avec ce composant. Le composant ne voit pas le onebox iframe. Vous devez coller le code iframe directement pour que le composant fonctionne. Et ce que personne ne fait, c’est que tout le monde met seulement l’URL, pas le code iframe.

2 « J'aime »

Oui, l’idéal serait que les utilisateurs collent simplement ce qui est fourni par la source et que cela fonctionne automatiquement dans Discourse !

Cela semble assez basique pour les débutants !

1 « J'aime »

Ah je vois, ça a du sens :+1:

Absolument pr-welcome si quelqu’un souhaite essayer de faire fonctionner ce composant de thème avec la nouvelle implémentation ‘lazy-videos’ du cœur.

4 « J'aime »

Merci encore. Le composant de thème Media Overlay de GitHub - davidtaylorhq/discourse-media-overlay-theme était la raison. Lorsque je l’ai désactivé, Discourse a recommencé à fonctionner correctement.

1 « J'aime »