Media Overlay component

مكون السمة
22

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)
25

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)
26

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)
27

يبدو أن مكون السمة هذا (الرائع جدًا) يعمل بالفعل، ولكنه مُصنف على أنه “معطل”.

لا يمكنني جعله لا يعمل.

هل يمكن للآخرين الاختبار أيضًا وتقديم النتائج؟

إذا لم يكن معطلاً، هل يمكننا إزالة علامة “معطل”؟

30

مرحباً @denvergeeks. يبدو أن هذا تم وضع علامة عليه على أنه معطل بسبب بعض المشكلات المتعلقة بمربعات YouTube. هل يمكنك تأكيد ما إذا كانت مقاطع فيديو YouTube تعمل بشكل جيد على موقعك مع مكون السمة هذا؟

32

شكراً @david – أنت على حق، فهو يرمي الأخطاء ويتعثر في مقاطع فيديو YouTube وإطارات iframe أيضًا، لكنني قمت بعمل fork وأزلت LazyYT في .js وهو يعمل بعد ذلك…

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

إعجابَين (2)
33

عظيم! هل تمانع في إنشاء طلب سحب (PR) بهذا التغيير؟

إعجاب واحد (1)
34

حسنًا، أعتقد أنني قمت بطلب السحب هذا بشكل صحيح (بما فيه الكفاية).

3 إعجابات
35

يبدو رائعًا، شكرًا لك. لقد قمت الآن بإزالة broken من هذا الموضوع :rocket:

3 إعجابات
36

حسناً هذا رائع! شكراً على الإصلاح يا @denvergeeks و @david :slight_smile:

إعجابَين (2)
37

لن يقوم كل مستخدم بتضمين رمز iframe، سيقوم الجميع بتضمين عنوان URL للفيديو مباشرة. ربما يجدر تنفيذ أداة لتحويل عنوان URL لفيديو YouTube إلى رمز تضمين iframe

38

أنا متأكد تمامًا من أنه يمكن إنشاء مكون سمة لهذا ووضع زر على شريط أدوات المنشئ. أو يمكن نسخ هذا الجزء مع إضافة هذا الجزء.

39

ستؤدي إضافة زر إضافي إلى تعقيد الواجهة. من الضروري أن يتم تحويل عناوين URL تلقائيًا، وبشكل غير محسوس للمستخدم. عندها ستعمل جميع مقاطع الفيديو المضمنة بشكل صحيح، بغض النظر عن طريقة التضمين.

إعجاب واحد (1)
40

حسناً. يمكن للمرء بعد ذلك إنشاء نسخة منه وجعلها تلقائية على ما أعتقد. :woman_shrugging:t2:

إعجابَين (2)
41

ربما يجب أن يكون في النواة بحيث يتم تحليل جميع عناوين URL بنفس المعيار؟

42

لست متأكدًا من فهمي للطلب هنا… يقوم Discourse بالفعل بتوسيع روابط YouTube إلى إطارات مضمنة كجزء من نظام “onebox” الخاص بنا. فقط الصق عنوان URL في سطر منفصل:

إعجابَين (2)
43

نعم، ولكن لصق الرابط مباشرة لا يعمل مع هذه المكونة. المكونة لا ترى الـ iframe onebox. تحتاج إلى لصق كود الـ iframe مباشرة لكي تعمل المكونة. وما لا يفعله أحد، هو أن الجميع يضعون الرابط فقط، وليس كود الـ iframe.

إعجابَين (2)
44

نعم، سيكون الوضع المثالي هو أن يقوم المستخدمون بلصق أي شيء يتم توفيره من المصدر ويعمل تلقائيًا في Discourse!

يبدو هذا أساسيًا جدًا للمبتدئين!

إعجاب واحد (1)
45

أرى ذلك، هذا منطقي :+1:

بالتأكيد pr-welcome إذا أراد أي شخص محاولة جعل مكون السمة هذا يعمل مع تطبيق ‘lazy-videos’ الجديد الأساسي

4 إعجابات
46

شكراً مرة أخرى. كان مكون سمة Media Overlay من GitHub - davidtaylorhq/discourse-media-overlay-theme هو السبب. عندما قمت بإيقافه، بدأ Discourse في العمل بشكل صحيح.

إعجاب واحد (1)