Skip to content
This repository has been archived by the owner on Jun 18, 2022. It is now read-only.

Dont't fix image with empty src #90

Open
bpahan opened this issue Aug 30, 2017 · 2 comments
Open

Dont't fix image with empty src #90

bpahan opened this issue Aug 30, 2017 · 2 comments

Comments

@bpahan
Copy link

bpahan commented Aug 30, 2017

I have faced with extra reqeust in IE and after investigation I noticed that ofi generates them.

I use tag image without src attribute at all (or it can be image with empty src) for lazy loading. In that case ofi process this image and create backgroun-image style with empty source which make new request to current page.

Could you suggest the desicion for this case?

Repository owner deleted a comment from hirbod Jan 14, 2018
@matteobad
Copy link

Hi @fregante, I saw that this bug is still open and that there is a brach with the fix in it. I and my team are using your awesome polyfill, but in order to make it work properly, we are currently applying a patch like the one you merged here.
We were wondering why this is not been merged in master and released with a new version.

Can I do something to help with it? Thanks 😁

@fregante
Copy link
Owner

fregante commented Oct 1, 2020

I haven't merged it because I don't have time to test it properly on IE and don't want to break it for anyone who has been using it unchanged for the past 3 years.

If you'd like to help, you can test it on IE9, IE11 and let me know if it works on both:

  • does the demo still work correctly in both browsers?
  • does it work if you add the src after?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants