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

Add YouTube embed FastHTML component #654

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dgwyer
Copy link
Contributor

@dgwyer dgwyer commented Feb 11, 2025


name: Pull Request
about: Propose changes to the codebase
title: '[PR] '
labels: ''
assignees: ''


Proposed Changes
Add a new custom core component to add a YouTube embedded video. It's basically a wrapper around the Iframe component.

Basic usage:

YouTubeEmbed("Auqrm7WFc0I")

Customisation options:

YouTubeEmbed("Auqrm7WFc0I", width=1120, height=630, start_time=60, no_controls=True)

Screenshot shows a demo with all customisation options used.
image

Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

  • New feature (non-breaking change which adds functionality)

Checklist
Go over all the following points, and put an x in all the boxes that apply:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I am aware that this is an nbdev project, and I have edited, cleaned, and synced the source notebooks instead of editing .py or .md files directly.

Copy link

gitnotebooks bot commented Feb 11, 2025

Found 1 changed notebook. Review the changes at https://app.gitnotebooks.com/AnswerDotAI/fasthtml/pull/654

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.

1 participant