[Poll] Optimizing the Q dependency

GuillaumeLeclerc commented 8 years ago

In order to reduce the build size and ship a library that will provide the best experience to your users. I'm currently trying to optimize the build of vue-google-maps.

I have two possible improvements in mind:

Use an alternative to Q promises, If you have a good suggestion of a lighter implementation of Kriskowal/Q promises
Inline Q inside vue-google-maps. Since the gzipped version of vue-google-maps is less than 25kb in the last v1.0 build. I was considering to Include Q inside vue-google-maps and does not have it as a dependency (only for the not bundled version). It would certainly increase the size for the ones already using Q. but since I think this not a huge proportion it should not matter. On the other side it has the advantage of saving one Http query. Moreover the gzip size of two files concatenated is usually lower than the sum of their gzip sized. Therefore it would reduce the bandwidth usage for most of us.

Tell me what you think

xkjyeah commented 8 years ago

Since we are already using Babel, why not use the built-in Promises library?

GuillaumeLeclerc commented 8 years ago

Does it support Deferred ? Because I know they are not part of the official Promise specification. As far as I understood the goal of babel is to implement next specifications of the language not more. I guess they didn't included Deferred

xkjyeah commented 8 years ago

Well, no, but it isn't hard to simulate Deferred:

function deferred() { var rv = {} rv.promise = new Promise((resolve, reject) => { rv.resolve = resolve; rv.reject = reject }) return rv }

On Thu, May 5, 2016 at 5:01 PM, Guillaume Leclerc notifications@github.com wrote:

Does it support Deferred ? Because I know they are not part of the official Promise specification. As far as I understood the goal of babel is to implement next specifications of the language not more. I guess they didn't included Deferred

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/GuillaumeLeclerc/vue-google-maps/issues/57#issuecomment-217106923

GuillaumeLeclerc / vue-google-maps

[Poll] Optimizing the Q dependency #57