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

No support for images? #119

Open
njaal opened this issue Sep 26, 2013 · 3 comments
Open

No support for images? #119

njaal opened this issue Sep 26, 2013 · 3 comments

Comments

@njaal
Copy link

njaal commented Sep 26, 2013

I've tried to use your plugin with only images as content, but it doesn't seem to work? I might be doing it wrong though, but couldn't seem to find any information if this is supposed to work or not.

I have been using the example provided, when I only have text or lists as content, it works. But when replacing with only images, it doesn't seem to work.

Here is an example:

skjermbilde 2013-09-26 kl 12 49 48

@frankie-loves-jesus
Copy link

Best set up a testcase at JSFiddle.

@jtara
Copy link
Member

jtara commented Oct 4, 2013

Images need a definite size set, using either the <image> tag width and height options, or else CSS width/height.

Otherwise, you need to arrange to call refresh once the images are all loaded.

I've used this plugin successfully for that purpose:

http://desandro.github.io/imagesloaded/

@jtara jtara closed this as completed Oct 4, 2013
@jtara
Copy link
Member

jtara commented Oct 4, 2013

Re-opening. The imagesloaded plugin should be mentioned in the documentation.

@jtara jtara reopened this Oct 4, 2013
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

3 participants