Soundcloud와 YouTube 원박스 썸네일 문제

제품 지원
2

토의한 대로, 썸네일 생성 로직이 이제 코어에 내장되었습니다.

TLP가 이전에 수행해야 했던 작업은 대부분 더 이상 필요하지 않습니다.

참고로, 포함을 위한 코어의 기준은 다음과 같습니다:

  def extract_images_for_post
    # src 속성이 있는 모든 이미지
    @doc.css("img[src]") -
    # 이모지 제외
    @doc.css("img.emoji") -
    # 인용문 안의 이미지 제외
    @doc.css(".quote img") -
    # 원박스 사이트 아이콘 제외
    @doc.css("img.site-icon") -
    # 원박스 아바타 제외
    @doc.css("img.onebox-avatar") -
    # 작은 원박스 이미지 제외 (큰 이미지는 .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") -
    # 원박스 사이트 아이콘 제외
    @doc.css("img.site-icon") -
    # 원박스 아바타 제외
    @doc.css("img.onebox-avatar") # Discourse Core보다 더 넓은 기준
  end

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

보시다시피, 코어에서 정의한 작은 원박스 이미지를 허용함으로써 제외 항목의 수를 줄이고 있습니다.

이를 TLP에서 유지보수하는 대신, 사용 사례 전반에 걸쳐 합의와 일관성을 도출하는 것이 좋겠습니다. 그래야 플러그인에서 이를 유지보수할 필요가 전혀 없게 됩니다.

이 사항은 @Arkshine 님에게도 관련이 있습니다.