Problèmes avec les vignettes des oneboxes Soundcloud et YouTube

Soutien
2

Comme discuté, la logique de génération des vignettes est désormais intégrée au cœur du système.

Ce que TLP devait faire n’est plus nécessaire, dans une large mesure.

Pour référence, les critères de base pour l’inclusion sont les suivants :

  def extract_images_for_post
    # toutes les images avec un attribut src
    @doc.css("img[src]") -
    # moins les émojis
    @doc.css("img.emoji") -
    # moins les images à l'intérieur des citations
    @doc.css(".quote img") -
    # moins les icônes de site onebox
    @doc.css("img.site-icon") -
    # moins les avatars onebox
    @doc.css("img.onebox-avatar") -
    # moins les petites images onebox (les grandes images sont .aspect-image-full-size)
    @doc.css(".onebox .aspect-image img")
  end

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

TLP modifie légèrement cela, mais uniquement pour assouplir les critères. J’aimerais pouvoir supprimer complètement cette surcharge si possible :

  def extract_images_for_post
    # toutes les images avec un attribut src
    @doc.css("img[src]") -
    # moins les émojis
    @doc.css("img.emoji") -
    # moins les images à l'intérieur des citations
    @doc.css(".quote img") -
    # moins les icônes de site onebox
    @doc.css("img.site-icon") -
    # moins les avatars onebox
    @doc.css("img.onebox-avatar") # Critères plus larges que le cœur de Discourse
  end

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

Comme vous pouvez le voir, je réduis le nombre d’exclusions en autorisant les petites images onebox telles que définies par le cœur du système.

Plutôt que de maintenir cela dans TLP, il serait préférable d’obtenir un consensus et une parité entre les cas d’utilisation, afin que je n’aie pas à maintenir cela dans le plugin du tout.

Cela vous concerne également @Arkshine