Skip to content

Support for switching to embedded players#6662

Closed
ChunkyProgrammer wants to merge 1 commit intoFreeTubeApp:developmentfrom
ChunkyProgrammer:embed-support
Closed

Support for switching to embedded players#6662
ChunkyProgrammer wants to merge 1 commit intoFreeTubeApp:developmentfrom
ChunkyProgrammer:embed-support

Conversation

@ChunkyProgrammer
Copy link
Member

Support for switching to embedded players

Pull Request Type

  • Feature Implementation

Related issue

#451

Description

A quick proof-of-concept that seems to work surprisingly well

Screenshots

FreeTube with a youtube embed (ads are blocked and the iframe is somewhat sandboxed, links to other videos + to the channel are blocked):
image

FreeTube with an invidious embed (dash not enabled):
image

Testing

Not including tests because I don't think should be merged unless absolutely necessary (which it isn't right now)

@github-actions github-actions bot added PR: waiting for review For PRs that are complete, tested, and ready for review and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Jan 28, 2025
@openforfuture
Copy link

Since YouTube occasionally changes its platform in ways that break FreeTube’s functionality, having embedded players as a fallback would be extremely useful. Currently, the only alternative is to open the embedded video in a browser (or in an external player that supports YouTube links, but when YouTube changes something, they often break as well). This workaround is also quite inconvenient, especially because video views aren’t tracked. If you want the video to appear greyed out and be recorded in your history, you have to do it manually.

So please consider implementing this feature — it would significantly improve the user experience. Nowadays, the embedded player is often the only way to watch content with fewer distractions. Thank you! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants