Giter Club home page Giter Club logo

Comments (5)

johnculviner avatar johnculviner commented on June 27, 2024

Thanks for the heads up. Someone did a pull to 'cleanup' the IFRAME but this was an unintended consequence. Would you like to do a pull or should I integrate your fix manually?

from jquery.filedownload.

vliatsos avatar vliatsos commented on June 27, 2024

Thank you for the plugin! Feel free to integrate the fix manually - it's only two characters in one line. My only concern is that 500 ms is somewhat arbitrary. Could there still be cases where the iframe is removed before the file has been downloaded? that's why I was interested to hear if there were better solutions out there.

from jquery.filedownload.

mcse3010 avatar mcse3010 commented on June 27, 2024

I plugged in 1000ms, and it didn't seem to have any negative effect, but
it did solve our problem...
On Aug 7, 2013 12:13 PM, "Vassilis Liatsos" [email protected]
wrote:

Thank you for the plugin! Feel free to integrate the fix manually - it's
only two characters in one line. My only concern is that 500 ms is somewhat
arbitrary. Could there still be cases where the iframe is removed before
the file has been downloaded? that's why I was interested to hear if there
were better solutions out there.


Reply to this email directly or view it on GitHubhttps://github.com//issues/36#issuecomment-22262565
.

from jquery.filedownload.

johnculviner avatar johnculviner commented on June 27, 2024

Yeah... anytime you do a setTimeout > 0 its often less than ideal. Maybe I'll just take the cleanup out all-together. Someone did a pull on it once and though it was messy to leave the iframe around... Which is better an iframe stuck in the DOM or the thing not working?? I think I know the answer...

from jquery.filedownload.

vliatsos avatar vliatsos commented on June 27, 2024

Agreed - the best solution for now would be to remove the lines in cleanUp() that remove the iframe to get the plugin back to a working state. I do not particularly like the proliferation of iframes in the DOM but that's the lesser evil. Let me know when it's fixed and I can close the issue.

from jquery.filedownload.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.