Skip to content

Use the YouTube embed screen in existing meeting files#2361

Open
JFWooten4 wants to merge 9 commits intostellar:mainfrom
JFWooten4:2145-pt-1
Open

Use the YouTube embed screen in existing meeting files#2361
JFWooten4 wants to merge 9 commits intostellar:mainfrom
JFWooten4:2145-pt-1

Conversation

@JFWooten4
Copy link
Copy Markdown
Contributor

Right now, there are a lot of old meetings that use the old iframe embed. This makes it tedious to update the display method over time. This PR migrates those old import schemes to use the new YouTube script added in #1876.

It also adds back controls to the player. In my original envisioning of the embed back in 2024, it seemed like just a useful tool to throw in some of these amazing Lumenaut videos. Since they are all under 7 or so minutes (and that's aggressive), I didn't think we needed controls.1 But now that I see it applies to hour-long meetings, we're clearly going to need those if we don't want readers to bounce over to YouTube and away from our docs.

In the future, I think it'd be clearly cool to add a mini player where the video can stay going in the bottom corner while the visitor explores other written pages in the docs.

Footnotes

  1. This has the added benefit of increasing watch time, based on my experiences in the past incentivizing pageview conversions.

Copilot AI review requested due to automatic review settings April 16, 2026 16:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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