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 Frostyx's youtube video to documentation page #137

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

xsuchy
Copy link
Member

@xsuchy xsuchy commented Jun 28, 2024

No description provided.

<div class="card">
<div class="card-body">
<h5 class="card-title">Demo with comments</h5>
<a target="_blank" rel="noopener noreferrer" href="https://www.youtube.com/watch?v=_-O7ryKCnlQ"><img src="/img/youtube-preview.jpg" alt="Demo video" style="max-width: 100%; display: inline-block;" /></a>
Copy link
Member

Choose a reason for hiding this comment

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

Did you intentionally not use the embed iframe or you don't know them?

If you go to a youtube video https://www.youtube.com/watch?v=_-O7ryKCnlQ and then click the "Share" button (next to likes and dislikes) and then click "Embed", it generates an <iframe> for you. Which is the way youtube videos are typically displayed on websites. See e.g. here https://frostyx.cz/posts/lsp-for-vim-boomers

It's easy to adjust the widht and height properties as needed.

I'd probably do it that way but if you intentionally wanted to do it with an image and link to youtube, I am okay with that as well.

Copy link
Member Author

Choose a reason for hiding this comment

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

I intentionally did not use it.

Copy link
Member

Choose a reason for hiding this comment

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

In that case +1 :-)

@jpodivin jpodivin merged commit 35b685d into fedora-copr:main Jul 4, 2024
6 checks passed
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.

3 participants