Problemi con le miniature di Soundcloud e YouTube onebox

Supporto
2

Come discusso, la logica di produzione delle miniature è ora integrata nel core.

Ciò che TLP doveva fare in passato non è più necessario, in gran parte.

A titolo di riferimento, i criteri principali per l’inclusione sono i seguenti:

  def extract_images_for_post
    # tutte le immagini con un attributo src
    @doc.css("img[src]") -
    # meno emoji
    @doc.css("img.emoji") -
    # meno immagini all'interno delle citazioni
    @doc.css(".quote img") -
    # meno icone di sito onebox
    @doc.css("img.site-icon") -
    # meno avatar onebox
    @doc.css("img.onebox-avatar") -
    # meno immagini onebox piccole (le immagini grandi sono .aspect-image-full-size)
    @doc.css(".onebox .aspect-image img")
  end

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

TLP modifica leggermente questo codice, ma solo per renderlo meno restrittivo. Vorrei eliminare completamente questa sovrascrittura, se possibile:

  def extract_images_for_post
    # tutte le immagini con un attributo src
    @doc.css("img[src]") -
    # meno emoji
    @doc.css("img.emoji") -
    # meno immagini all'interno delle citazioni
    @doc.css(".quote img") -
    # meno icone di sito onebox
    @doc.css("img.site-icon") -
    # meno avatar onebox
    @doc.css("img.onebox-avatar") # Criteri più ampi rispetto al Discourse Core
  end

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

Come puoi vedere, sto riducendo il numero di esclusioni consentendo immagini onebox piccole come definite dal core.

Invece di mantenere questo codice in TLP, sarebbe utile raggiungere un consenso e una parità tra i casi d’uso, in modo da non doverlo più mantenere nel plugin.

Questo è anche rilevante per te @Arkshine