Problemas con miniaturas de Soundcloud y YouTube oneboxes

Soporte
2

Como se discutió, la lógica de generación de miniaturas ahora está integrada en el núcleo.

Lo que TLP solía tener que hacer ya no es necesario, en gran medida.

Como referencia, los criterios principales para la inclusión son los siguientes:

  def extract_images_for_post
    # todas las imágenes con un atributo src
    @doc.css("img[src]") -
    # menos emojis
    @doc.css("img.emoji") -
    # menos imágenes dentro de citas
    @doc.css(".quote img") -
    # menos iconos de sitio de onebox
    @doc.css("img.site-icon") -
    # menos avatares de onebox
    @doc.css("img.onebox-avatar") -
    # menos imágenes pequeñas de onebox (las imágenes grandes son .aspect-image-full-size)
    @doc.css(".onebox .aspect-image img")
  end

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

TLP modifica esto ligeramente, pero solo para relajarlo. Me gustaría eliminar esta sobrescritura por completo si es posible:

  def extract_images_for_post
    # todas las imágenes con un atributo src
    @doc.css("img[src]") -
    # menos emojis
    @doc.css("img.emoji") -
    # menos imágenes dentro de citas
    @doc.css(".quote img") -
    # menos iconos de sitio de onebox
    @doc.css("img.site-icon") -
    # menos avatares de onebox
    @doc.css("img.onebox-avatar") # Criterios más amplios que el núcleo de Discourse
  end

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

Como pueden ver, estoy reduciendo la cantidad de exclusiones permitiendo imágenes pequeñas de onebox según lo definido por el núcleo.

En lugar de mantener esto en TLP, sería bueno lograr un consenso y paridad en todos los casos de uso, para no tener que mantenerlo en absoluto en el plugin.

Esto también es relevante para ti @Arkshine