-
Notifications
You must be signed in to change notification settings - Fork 150
question: Order of polyfills.js #126
Comments
I am also interested in the answer and if this appears to be safe thing to do, it would be great if Ionic's base project could be updated to include it. This issue took me a lot of time and seems fixed now, so it's best to be initially fixed for other people in the base project. Thanks! |
I would know this as well. Thanks! |
Anyone out there? |
Hi, I found angular/zone.js/868 and I can't find a way to use @JiaLiPassion cordova patch properly. If anyone can help it would be great. Thank you |
@paulsouche , if it is the same issue which angular/zone.js#868 resolved, you could just use the newest version of |
@JiaLiPassion I'm already on BTW I think that cordova FileReader should not be global and we should not have this conversation but @joewoodhouse's workaround is more a hack than something reliable and that's why I'm asking for help of somebody who know more about Thank you for the great job anyway |
The background to this question comes from danielsogl/awesome-cordova-plugins#505
Basically it appears there's an issue with how the FileReader API is patched by various agents, namely Zone.js, Ionic Native and the Cordova file plugin itself. The resulting behaviour made events like
onloadend
and similar not reliably fire from the FileReader. It pointed to a Zone.js related issue.After a bit of exploring and experimenting I found that by moving the
polyfills.js
script tag to the top of myindex.html
, before thecordova.js
tag, the issue went away. Presumably this is becausepolyfills.js
containszone.js
in it, and so by this re-ordering somehow the issue is resolved.It would be good to know if this is a safe thing to do. Presumably there was a reason why the polyfills.js was put after cordova.js originally.
Any information much appreciated.
The text was updated successfully, but these errors were encountered: