-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[APP-932] Additional requested embed players #2314
Comments
Sketchfab embeds would be useful for 3D artists. |
As a side-note. The app currently automatically opts-in users to this functionality. However, from a data privacy perspective this is kind of edgy since the app now per default (at least) leaks the user's IP address to 3rd parties like Google, Meta, etc. pp. |
Another thing to add to the list, handle the youtube mobile links that prefix with |
It only contacts those services if the users tap into it and I think people generally understand that those services have to be contacted to display content from them |
It really depends on the market you're in. I'm coming from a European / GDPR centric POV where you'd have to have the user opt-in to this feature, even if you're only running it when the user clicks on it. But that's only for Europe (and California iirc). |
linking @seboslaw's post about the solution to that https://bsky.app/profile/seabass.bsky.social/post/3khhzleejky2a
Frankly might be a good idea anyway just to keep app review off of you. |
If there's a regulatory requirement we should do it on first tap I figure. If you have to go into settings and enable it then nobody will. |
Agreed. It's non-obvious to go into settings to do it. Probably just display a sheet on first press letting the user know, then remember the setting from there. Also to be fair, even without a regulation, I think people are attracted to the "less intrusive" aspect of federated stuff and giving warning is a good faith sort of thing. |
Uh, so I have the exact opposite request, I'm currently seeing gray rectangles in place of thumbnails until I click on the box, and I want to see the thumbnail immediately ;) |
yeah something seems to be inconsistent about the metadata fetch now, I gotta look into that |
As well as a short explanation shown on first tap, it might be a good idea to add a section to your Privacy Policy explaining the behaviour? https://blueskyweb.xyz/support/privacy-policy e.g. perhaps by updating and expanding this section: |
Opened an issue on the interstitial #2338 |
A few things of note while implementing these.
|
As someone that's been active on imgur for over a decade, before giphy/tenor took over it was THE place (and still is for certain styles/eras) to post/store/share animated meme images, often tailored ones or highly customized ones, as well as supporting much higher resolutions than most other options, 20MB for static images, and up to 10k on a side, thus the source of the 'intergalactic' quality image macros. It lacks any good search tools (lack of text being added to most posts, and the tag system being too limited and itself meme-ified as a result), but folks build extensive bookmarks/folder chains on-site and share them around to catalog images, it's basically the source of "Nice Meme, yoink!" type image-macros, etc. It also has a pretty extensive 'directly import video from other sites' toolkit built right in for making clips to turn into memes/image macros, etc. It might be a little niche, but supporting directly embedding imgur content would be convenient for a lot of the reasons folks want to be able to post videos (yes, for the memes and other short clips), if you haven't been active on imgur across that whole decade-plus existence it can be an EXTREMELY opaque culture/site to explore due to how much of it is cyclically tied to specific dates as well for when certain content does/doesn't exist, etc. Edit: One minor addendum also, Imgur's API does let you check if a given image is animated or not, and they've never supported anything except gifv or mp4 for video. Webp is just "better jpeg" for that site. The bandwidth/processing costs to convert everything to webm instead of just serving mp4 was enough they decided to just keep serving mp4 and gif as a fallback. api.imgur.com/3/image/UID, with UID being the "BXsPWci" out of "https://i.imgur.com/BXsPWci.gif" for example. Requires a header of "Authorization: Client-ID CID" with CID being a one-time client app registration. NOT a login against the site, no oauth required since it's just requesting public image data, so it'd just be "bluesky" globally or per-app or however you folks decided to organize things. https://apidocs.imgur.com/#authorization-and-oauth goes into that a bit more. |
Does Bluesky's YouTube embeds currently handle Playlist links? I'd also like to request:
|
YT playlists doable. Bandcamp embeds are tricky, would need to play with it and see what works. This is the embed URL: https://bandcamp.com/EmbeddedPlayer/album=3394138826/size=large/bgcol=ffffff/linkcol=0687f5/tracklist=false/artwork=small/track=3559172457/transparent=true/. The URL that you'd be sharing is: https://homesick89.bandcamp.com/track/1000db-stare. Notice that the embed URL is using an album ID while the share link is not. I'm not certain how we'd be able to get that or if it is possible. Have you seen anywhere else where these embeds are automatically taken care of? For sketchfab, it's doable, but would want to see what the performance is like in a webview. Streamable is actually super easy, and I think it is my preferred embed right now. Best video experience of the options. There's a blocker though with retrieving the thumbnails for them. I'll open a separate issue on that this week and hopefully it can be sorted out separately. |
Fortunately it isn't so much an issue of being able to get the info from the API when loading embeds (this wouldn't work anyway, their rate limit is far too aggressive for this purpose). If we were always loading the content from the start, we wouldn't have any issue. Instead, we first load a placeholder that is always not animated, and we have no information about whether it is or not. We'd probably need to make a request to the API endpoint you mentioned when posting the image and include info in the embed about whether it is animated or not. That's outside the scope of my current PR for various reasons (adding external APIs, modifying the embed metadata behavior, etc) so I can't really do it. But regardless I do agree they would be nice! I suppose we could always just display the gif player even before we know if it is a gif or not, but that seems like a weird UX if you're thinking you're going to be playing a gif but then it doesn't animate... |
Thankfully, it's all in the pages' https://homesick89.bandcamp.com/album/exitminilp007-homesick-1000db-stare-mini-lp https://homesick89.bandcamp.com/track/1000db-stare
Additionally, embeddable player URLs are provided upfront under
(note: This embeddable URL is easy to manipulate and customize. For example, here's an album player with large artwork, tracklist enabled, a dark background and yellow links. The order of parameters doesn't appear to matter.
Tumblr, Twitter, and Cohost. I believe Tumblr uses the Twitter's embedded player seems to differ slightly from the Both appear to use Bandcamp's "Artwork-only" player. |
@HeyItsLollie yea i've opened an issue here about that: bluesky-social/atproto#2009 right now there's no where to put that info nor is it provided in the metadata fetch response. if both of those are done, then we can really make this work! (and afaik both are rather trivial to do). but the lack of both of those blocks this for the moment. we literally just had a thread about this on bsky. thanks for the great stuff! |
Actually a bit more digging... ...turns out they DO indicate the difference in the OpenGraph details. https://www.opengraph.xyz/url/https%3A%2F%2Fimgur.com%2FTgdRHOd (still image) https://www.opengraph.xyz/url/https%3A%2F%2Fimgur.com%2FBXsPWci (animated image) Animated images always get a .gif for the twitter:image and og:image parameters, and likewise still images never do. Avoids the API rate limiting and since opengraph is used for so much embedding details imgur exposing animated vs not via that rubrik should be usable I think? Maybe in the other issue you opened instead of this one, I'm good at tackling API nonsense but I admit I don't feel up to building the BSky app to test ideas directly in the codebase. |
Yea its 100% workable with that data, but just needs some changes both to the |
If this is the right place to ask for additional players - would you consider adding YouTube Music https://music.youtube.com ? |
Please add support for links such as https://packaged-media.redd.it/tajnlkezevdc1/pb/m2-res_472p.mp4?m=DASHPlaylist.mpd&v=1&e=1705950000&s=a066af59715c9eb450f69037e71c95541aac2565#t=0. |
I can already say there's no way Bluesky will implement embeds for direct links to video files. Why would you want to link to this file directly, instead of the Reddit post that this video originates from? |
Because that is complicated. I take it that in order for video support to work, it has to be embedded from a different site which has to be allowed by the developers for the code to be added or hosted directly on one of bsky.app's servers. |
I'd like to suggest a static fallback for sites like TikTok whose embed are a bit trickier. Currently, including a link card from Tiktok just shows the root site's title, which isn't especially informative. Instead, Tiktok's oembed response (docs here) provides:
How about at least using that info for a static card that links to the video? |
We can anything that's secure and has moderation on their platform!
APP-932
The text was updated successfully, but these errors were encountered: