Проблемы с миниатюрами из однобоксов Soundcloud и YouTube

Поддержка
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.