Problemas com miniaturas dos oneboxes do Soundcloud e do YouTube

Suporte
2

Como discutido, a lógica de geração de miniaturas (thumbnailing) já está integrada ao núcleo.

O que o TLP precisava fazer antes não é mais necessário, em grande parte.

Como referência, os critérios principais para inclusão são os seguintes:

  def extract_images_for_post
    # todas as imagens com um atributo src
    @doc.css("img[src]") -
    # menos emojis
    @doc.css("img.emoji") -
    # menos imagens dentro de citações
    @doc.css(".quote img") -
    # menos ícones de site de onebox
    @doc.css("img.site-icon") -
    # menos avatares de onebox
    @doc.css("img.onebox-avatar") -
    # menos imagens pequenas de onebox (imagens grandes são .aspect-image-full-size)
    @doc.css(".onebox .aspect-image img")
  end

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

O TLP modifica isso ligeiramente, mas apenas para flexibilizar. Gostaria de eliminar completamente essa sobrescrita, se possível:

  def extract_images_for_post
    # todas as imagens com um atributo src
    @doc.css("img[src]") -
    # menos emojis
    @doc.css("img.emoji") -
    # menos imagens dentro de citações
    @doc.css(".quote img") -
    # menos ícones de site de onebox
    @doc.css("img.site-icon") -
    # menos avatares de onebox
    @doc.css("img.onebox-avatar") #Critérios mais amplos que o Núcleo do Discourse
  end

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

Como você pode ver, estou reduzindo a quantidade de exclusões ao permitir imagens pequenas de onebox conforme definido pelo núcleo.

Em vez de manter isso no TLP, seria bom obter consenso e paridade entre os casos de uso, para que eu não precise manter isso no plugin de forma alguma.

Isso também é relevante para você @Arkshine