Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

URL preview for messages in public rooms that allow guests #25

Open
almereyda opened this issue Jun 22, 2017 · 10 comments
Open

URL preview for messages in public rooms that allow guests #25

almereyda opened this issue Jun 22, 2017 · 10 comments
Labels
matrix-two Work on the redesign of matrix.to

Comments

@almereyda
Copy link

Currently when pasting a permanent link generated from Riot into a channel, there is no URL preview available when wanting to forward a message into another room.

Tested with

https://matrix.to/#/!JxSVcUkWaKRiETPsxS:matrix.allmende.io/$1498108097390ttXFy:matrix.allmende.io

which is locally expanded to

https://riot.allmende.io/#/room/!JxSVcUkWaKRiETPsxS:matrix.allmende.io/$1498108097390ttXFy:matrix.allmende.io
@uhoreg
Copy link
Member

uhoreg commented Jun 27, 2017

Maybe some way to hook into riot-static? @t3chguy, what do you think?

@t3chguy
Copy link
Member

t3chguy commented Jun 30, 2017

Previewing a Public Room in general, would be nice in matrix.to
Previewing a specific message, maybe in the future @uhoreg

@uhoreg
Copy link
Member

uhoreg commented Jun 30, 2017

Ah, I completely missed that this was about individual messages.

@t3chguy
Copy link
Member

t3chguy commented Jun 30, 2017

I guess it could be a hybrid, preview the public room and if the event is in memory highlight it
whatcha think @uhoreg ?

@uhoreg
Copy link
Member

uhoreg commented Jun 30, 2017

That might work, if you made it clear, when the event wasn't in memory, that it was just a preview of the whole room. Otherwise it may be confusing for users.

@t3chguy
Copy link
Member

t3chguy commented Sep 18, 2017

@uhoreg upon re-reading this seems to be about a URL Preview of matrix.to&the room it points to in Riot, this is not possible unless Synapse grows a thing that executes Javascript as matrix.to is entirely clientside

@MurzNN
Copy link

MurzNN commented Sep 21, 2017

At now we have already https://view.matrix.org/ that can do a static preview of most rooms, for example https://view.matrix.org/room/!cURbafjkfsMDVwdRDQ:matrix.org/ so now we can at first easily integrate iframe to matrix.to web pages.

Seems Matrix Static at now can't display specific message separately on page, but I think this is not hard to implement, so I post feature request: matrix-org/matrix-static#49

@JorikSchellekens
Copy link
Contributor

Rooms are now preview-able on matrix.to
Events are pending: matrix-org/matrix-spec-proposals#2644

@JorikSchellekens JorikSchellekens added the matrix-two Work on the redesign of matrix.to label Sep 21, 2020
@t3chguy
Copy link
Member

t3chguy commented Jul 15, 2021

Strictly speaking

URL preview for messages in public rooms that allow guests

Will not be possible, given that a room allowing guests would mean that m.to has to create a guest account and join the room

URL preview for messages in PEEKABLE rooms

would be achievable

@clokep
Copy link
Member

clokep commented Mar 29, 2022

URL preview for messages in PEEKABLE rooms

would be achievable

I was hoping this would be an easy-ish thing to knock out one day while off-hours since this really annoys me each time I post a matrix.to link. It seems that matrix.to currently loads information about the room in the background though, so it wouldn't be easy to embed any Open Graph info into the page itself.

One option could be to embed an oEmbed autodiscovery link to a JSON endpoint which returns some information though. This would at least work for Synapse to generate better previews.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
matrix-two Work on the redesign of matrix.to
Projects
None yet
Development

No branches or pull requests

6 participants