Probleme mit Thumbnails von Soundcloud und YouTube Oneboxes

Unterstützung
2

Wie besprochen, ist die Logik für die Erstellung von Vorschaubildern nun im Kern integriert.

Was TLP früher tun musste, ist größtenteils nicht mehr erforderlich.

Zur Referenz lauten die Kernkriterien für die Einbeziehung wie folgt:

  def extract_images_for_post
    # alle Bilder mit einem src-Attribut
    @doc.css("img[src]") -
    # abzüglich Emojis
    @doc.css("img.emoji") -
    # abzüglich Bilder innerhalb von Zitaten
    @doc.css(".quote img") -
    # abzüglich Onebox-Sitesymbole
    @doc.css("img.site-icon") -
    # abzüglich Onebox-Avatare
    @doc.css("img.onebox-avatar") -
    # abzüglich kleiner Onebox-Bilder (große Bilder sind .aspect-image-full-size)
    @doc.css(".onebox .aspect-image img")
  end

aus discourse/lib/cooked_post_processor.rb at main · discourse/discourse · GitHub

TLP passt dies leicht an, aber nur, um die Kriterien zu lockern. Ich würde diese Überschreibung gerne vollständig entfernen, wenn möglich:

  def extract_images_for_post
    # alle Bilder mit einem src-Attribut
    @doc.css("img[src]") -
    # abzüglich Emojis
    @doc.css("img.emoji") -
    # abzüglich Bilder innerhalb von Zitaten
    @doc.css(".quote img") -
    # abzüglich Onebox-Sitesymbole
    @doc.css("img.site-icon") -
    # abzüglich Onebox-Avatare
    @doc.css("img.onebox-avatar") #Breitere Kriterien als Discourse Core
  end

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

Wie Sie sehen, reduziere ich die Anzahl der Ausschlüsse, indem ich kleine Onebox-Bilder zulasse, wie vom Kern definiert.

Anstatt dies in TLP zu pflegen, wäre es gut, Konsens und Parität über die Anwendungsfälle hinweg zu erreichen, damit ich dies überhaupt nicht im Plugin pflegen muss.

Dies ist auch für Sie relevant, @Arkshine