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

[Info] Works perfectly with picturefill.js #51

Open
akroii opened this issue Apr 1, 2016 · 2 comments
Open

[Info] Works perfectly with picturefill.js #51

akroii opened this issue Apr 1, 2016 · 2 comments

Comments

@akroii
Copy link

akroii commented Apr 1, 2016

Try

<div class="full focuspoint" data-focus-x="-0.02" data-focus-y="-0.1" data-image-w="2250" data-image-h="1992">
<picture>
<!--[if IE 9]><video style="display: none;"><![endif]-->
<source srcset="img/yyy1.jpg" media="(min-width: 1000px)">
<source srcset="img/yyy2.jpg" media="(min-width: 600px)">
<source srcset="img/yyy3.jpg" media="(min-width: 320px)">
<!--[if IE 9]></video><![endif]-->
<img src="img/picture.jpg" alt="">
</picture>
</div>

Nice ;)

@akroii
Copy link
Author

akroii commented Apr 16, 2016

... but it not work with <img srcset="..."

@yeldiRium
Copy link

... but it not work with <img srcset="..."

This is actually a problem, since using src can lead to double downloads with picturefill. Although this isn't really an issue with focus point, it would be nice to have the compatibility.

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

No branches or pull requests

2 participants