Limit the number of URL previews.

[turt2live]

Description

https://matrix.to/#/!UcYsUzyxTGDxLBEvLz:matrix.org/$1508940221476bTcDp:berigora.net
(#offtopic)

Version information

• Platform: web (in-browser)
• Browser: Chrome 62
• OS: Windows 10
• URL: riot.im/develop

[lampholder]

😱 🙀 amazing 😬

Yeah, I guess limiting it to... actually maybe limiting it to one would be good (and easy to remember). Maybe something like:

(That's actually ugly as hell, but, you know, something like that). It could even optionally click to expand.

[turt2live]

I kinda don't mind that, would something like this also work?

(please ignore the fact that I was too lazy to get the color code for riot text - it should be Riot green)

Showing 2 previews seems reasonable, as it isn't too bad on the timeline. 3 would probably be too many though. The only other significant things I changed were the text (to give the user a hint as to what clicking it does) and disconnected the left edge from the preview itself.

[lampholder]

Yeah, I like that. The reason I chose one rather than more-than-one was because I thought it would be super easy for people to remember - if they want to post three links with URL previews that's going to be three individual messages.

I'm not super wedded to that though - anything which puts a reasonable limit on it and lets people expand to see the full list is good I think.

The risk with expanding/contracting timeline items is that they leave you in a place in the timeline tha you didn't expect to be pre/post expansion/contraction - if we can impl a solution for this without falling into that trap, I think this would be a good thing to do :)

[turt2live]

Updated original post for the bounty :)

[t3chguy]

I think I'll take this on, I personally prefer 2 but I'll do it in a way where the number is just a const atop the file for easy tweaking l8r