Non si basa sulle dimensioni dell’immagine. In generale, non utilizziamo le immagini dalle onebox, poiché in molti casi non sono adatte per le miniature. Ad esempio, su Meta vediamo spesso link a repository GitHub come
Nessuno vuole che la mia faccia venga ingrandita e utilizzata come miniature del topic
Beh, è molto specifico per Meta: nella nostra community non abbiamo collegamenti a GitHub. Si tratta di notizie e dovrebbero essere utilizzate le immagini onebox. Questo limita molto le miniature, proprio a causa di questo problema del “volto” su Meta. Esiste un modo per modificarlo e aggiungere immagini dalle onebox? Immagino che molte community non diano importanza ai “volti” di GitHub.
Sono certamente aperto a miglioramenti in questo ambito. So che anche @merefield è interessato a questa logica. La modifica sarà necessaria nel core, non in un plugin, quindi la separerò in un nuovo argomento.
Ecco la logica esistente:
Forse potremmo aggiungere una classe .no-thumbnail a certi tipi di onebox (ad esempio GitHub) e istruire Discourse a ignorare solo quelli, mantenendo le immagini degli altri onebox.
Aggiungerò l’etichetta pr-welcome a questo argomento, ma assicurati di delineare un piano qui prima di intraprendere lavori sostanziali.
Sarebbe fantastico. Attualmente il 90% dei nostri argomenti non ha miniature, anche se potrebbe averle. Come detto, questa issue su GitHub rappresenta un caso limite molto specifico per le comunità di sviluppatori; la stragrande maggioranza di noi non ottiene miniature per argomenti che potrebbero certamente averle a causa di questo.
Secondo anche io!!! Per un caso d’uso come un aggregatore di notizie o qualsiasi altra pagina ricca di repost, non avere miniature pertinenti è un grosso problema.
Se non integrate questa funzionalità a breve, avete qualche idea su come farlo sulla mia distribuzione auto-ospitata?
Sì, al momento non è all’ordine del giorno perché le “facce” da GitHub potrebbero essere visualizzate in community di sviluppatori come Meta, ma la maggior parte delle community non si cura delle “facce” e desidera avere il maggior numero possibile di miniature, in particolare quando sono disponibili immagini onebox sufficientemente grandi per l’OP.
def extract_images_for_post
# tutte le immagini con un attributo src
@doc.css("img[src]") -
# meno le emoji
@doc.css("img.emoji") -
# meno le immagini all'interno delle citazioni
@doc.css(".quote img") -
# meno le icone dei siti onebox
@doc.css("img.site-icon") -
# meno gli avatar onebox
@doc.css("img.onebox-avatar") # Criteri più ampi rispetto al Discourse Core
end
Sì, grazie @merefield, ma usiamo solo il componente del tema di @david, non il TLP TC. Tuttavia, probabilmente userò la tua valutazione per applicare una patch al core se non vedremo alcun movimento o un’opzione fornita dal @team su questo problema.
Compreso. Al momento ho solo un semplice ambiente di sviluppo Docker per creare alcuni plugin semplici e patch core. Attenderò che qualcuno esamini la questione e, nel frattempo, probabilmente applicherà una patch tramite la tua valutazione.
Un lavoro adeguato qui non è un progetto immateriale, perché è necessario modificare i casi di test e lavorare su due componenti (Discourse e Discourse One-Box). Ma ne vale sicuramente la pena!
Per inciso, Ruby non è nulla di cui aver paura dopo Ember
Come soluzione temporanea, spesso copio e incollo manualmente l’immagine in una chat Whisper, quindi seleziono la miniatura da lì.
Sarebbe più comodo se fosse automatico, magari con un’opzione di configurazione per specificare i domini da cui non estrarre miniature (ad esempio per coprire il caso di GitHub).
Concordo. Abbiamo usato lo snippet di @merefield per sovrascrivere il core con un mini plugin. Ma dovrebbe essere facile per il team del core proporre qualcosa di pronto all’uso. Non averlo solo perché alcuni hanno paura di mostrare i loro volti su GitHub è semplicemente pigro.