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

Gallery: Add possibility to use iframes in slides #170

Merged
merged 1 commit into from
Dec 8, 2014

Conversation

drublic
Copy link
Owner

@drublic drublic commented Dec 3, 2014

A gallery can now handle iframes: Just provide a link to the according
iframe in data-src-fullsize instead of an image and set data-iframe
on the enclosing link.

<a href="#modal-gallery/iframe" data-iframe>
    <img alt="" src="image.jpg" data-src-fullsize="https://iframe-url.com/">
</a>

This adds functionality for videos from Vimeo and Youtube. They are not
paused when skipping to the next/prev element. Please consider adding a
custom script for handling this behavior.

The defined max-width of an iframe is currently 600px. You can change
this by setting a variable $gallery--iframe-size which overwrites the
current default value.

Please have a look, @Melros.

A gallery can now handle iframes: Just provide a link to the according
iframe in `data-src-fullsize` instead of an image and set `data-iframe`
on the enclosing link.

    <a href="#modal-gallery/iframe" data-iframe>
        <img alt="" src="image.jpg" data-src-fullsize="https://iframe-url.com/">
    </a>

This adds functionality for videos from Vimeo and Youtube. They are not
paused when skipping to the next/prev element. Please consider adding a
custom script for handling this behavior.

The defined max-width of an iframe is currently 600px. You can change
this by setting a variable `$gallery--iframe-size` which overwrites the
current default value.
@drublic drublic added this to the v1.3.0 milestone Dec 4, 2014
@drublic drublic merged commit f449148 into master Dec 8, 2014
@drublic drublic deleted the feature/video-gallery branch December 8, 2014 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant