d->m: Tenor GIFs from GIF button are too verbose #30

Closed
opened 2024-01-13 10:47:15 +00:00 by cadence · 2 comments
Owner

On Discord, the GIF button just sends a Tenor link to the chat.

Discord displays this by hiding the link (a recent change) and displaying the GIF in video format due to the server-generated URL preview embed (embed type: video). Link: 1195587910

Matrix displays this as a link with a URL preview where the URL preview does not animate because Synapse's thumbnailer sucks. Link: https://matrix.to/#/!TqlyQmifxGUggEmdBN:cadence.moe/$hwUkiPp0w2JNXwGdg2Rt07DoZPup8b34s0C6az4izeg?via=cadence.moe&via=matrix.org&via=conduit.rory.gay

(But who knows, maybe Synapse will get a better thumbnailer soon...)

Then, OOYE sees Discord's server-generated video embed, and bridges the video embed to Matrix as well, but this doesn't look very good at all. Link: https://matrix.to/#/!TqlyQmifxGUggEmdBN:cadence.moe/$VAk5MSNGlfopAlCmyYXgfcJtngSadxKoBXkpoFSia7s?via=cadence.moe&via=matrix.org&via=conduit.rory.gay

IDEAL: Matrix users should just see something animated. We should add a special case for Tenor URLs to do the right thing.

On Discord, the GIF button just sends a Tenor link to the chat. Discord displays this by hiding the link (a recent change) and displaying the GIF in video format due to the server-generated URL preview embed (embed type: video). Link: https://discord.com/channels/1160893336324931584/1160894080998461480/1195587910817300540 Matrix displays this as a link with a URL preview where the URL preview does not animate because Synapse's thumbnailer sucks. Link: https://matrix.to/#/!TqlyQmifxGUggEmdBN:cadence.moe/$hwUkiPp0w2JNXwGdg2Rt07DoZPup8b34s0C6az4izeg?via=cadence.moe&via=matrix.org&via=conduit.rory.gay (But who knows, [maybe Synapse will get a better thumbnailer soon...](https://github.com/matrix-org/matrix-spec-proposals/pull/2705)) Then, OOYE sees Discord's server-generated video embed, and bridges the video embed to Matrix as well, but this doesn't look very good at all. Link: https://matrix.to/#/!TqlyQmifxGUggEmdBN:cadence.moe/$VAk5MSNGlfopAlCmyYXgfcJtngSadxKoBXkpoFSia7s?via=cadence.moe&via=matrix.org&via=conduit.rory.gay IDEAL: Matrix users should just see something animated. We should add a special case for Tenor URLs to do the right thing.
Author
Owner

This is mostly fixed with better video embed rendering, but it would still be better if it was less verbose.

This is mostly fixed with better video embed rendering, but it would still be better if it was less verbose.
Author
Owner

Made it a bit smaller in edf60bcd2d. Good enough.

Made it a bit smaller in edf60bcd2d4c820c2e8eeac87da6771239c40551. Good enough.
Sign in to join this conversation.
No labels
blocking
web
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: cadence/out-of-your-element#30
No description provided.