مشاكل مع الصور المصغرة من Soundcloud و YouTube oneboxes

الدعم
2

كما تم الاتفاق، أصبحت منطقية إنشاء الصور المصغرة مدمجة الآن في النواة.

ما كان على TLP القيام به لم يعد ضروريًا إلى حد كبير.

للمرجعية، المعايير الأساسية للإدراج هي التالية:

  def extract_images_for_post
    # جميع الصور التي تحتوي على سمة src
    @doc.css("img[src]") -
    # ناقص الرموز التعبيرية
    @doc.css("img.emoji") -
    # ناقص الصور داخل الاقتباسات
    @doc.css(".quote img") -
    # ناقص أيقونات مواقع onebox
    @doc.css("img.site-icon") -
    # ناقص صور رموز onebox
    @doc.css("img.onebox-avatar") -
    # ناقص صور onebox الصغيرة (الصور الكبيرة هي .aspect-image-full-size)
    @doc.css(".onebox .aspect-image img")
  end

من discourse/lib/cooked_post_processor.rb at main · discourse/discourse · GitHub

يقوم TLP بتعديل هذا قليلاً، ولكن فقط لتوسيع نطاقه. أود التخلي عن هذا التجاوز بالكامل إذا أمكن:

  def extract_images_for_post
    # جميع الصور التي تحتوي على سمة src
    @doc.css("img[src]") -
    # ناقص الرموز التعبيرية
    @doc.css("img.emoji") -
    # ناقص الصور داخل الاقتباسات
    @doc.css(".quote img") -
    # ناقص أيقونات مواقع onebox
    @doc.css("img.site-icon") -
    # ناقص صور رموز onebox
    @doc.css("img.onebox-avatar") #معايير أوسع من نواة Discourse
  end

من https://github.com/paviliondev/discourse-topic-previews/blob/master/lib/cooked_post_processor_edits.rb

كما ترون، أنا أقوم بتقليل عدد الاستثناءات من خلال السماح بصور onebox الصغيرة كما عرّفتها النواة.

بدلاً من الحفاظ على هذا في TLP، سيكون من الجيد الحصول على توافق في الآلات وتكافؤ عبر حالات الاستخدام، حتى لا أضطر إلى الحفاظ على هذا في الإضافة على الإطلاق.

هذا الأمر ذو صلة أيضًا بك @Arkshine