Stable Version v0.1.0 discussion board about snap.svg.zpd HOT 9 CLOSED

For these lines
https://github.com/huei90/snap.svg.zpd/blob/beta_v0.1.0/snap.svg.zpd.beta.js#L317
https://github.com/huei90/snap.svg.zpd/blob/beta_v0.1.0/snap.svg.zpd.beta.js#L328
https://github.com/huei90/snap.svg.zpd/blob/beta_v0.1.0/snap.svg.zpd.beta.js#L341
and others

I prefer the original version one using like zpd('destroy')
to prevent define more and more namespace. I think using only one namespace zpd is enough.

We need to decrease the number of using namespace. Prevent more conflict with other plugin.

from snap.svg.zpd.

Yepp.. might be better. The new structure will though require at least two properties on the paper element (one for zpd functions, one for zpd references)

from snap.svg.zpd.

@tikiatua any new update?

from snap.svg.zpd.

Hi Huei,

sorry, did not get the change to work on the library any more as I was pulled into a new project. But I will try to find some time in december.

Greetings,
Ramon

from snap.svg.zpd.

Hi @tikiatua Since the Github Traffic Graph shows that there're still lot of viewing/searching on this project. Do you still have time on developing it?

from snap.svg.zpd.

Reference #18

from snap.svg.zpd.

nudge @tikiatua Need your reply.

from snap.svg.zpd.

Hi @huei90,

I am currently very busy on a few other projects but managed to negotiate a few days to spend on the zpd plugin in January. I think we should work on the api interface for version 1.0 first and then implement the functionality accordingly.

It might also be helpful to change the structure of the plugin to work in a more event based way, as this would make it easier to integrate multiple interaction possibilities (i.e. sliders, touch events, device movements).

I would propose to attach a zpd property to the Paper prototype and implement the functions as subproperties of this zpd element. This would also allow for easier extension of the plugins functionality with additional methods. A drawback would be that it breaks backward compatibility (but this is allowed for major version changes).

Paper.prototype.zpd = {
  init: function(){..},
  enable: function(){..},
  disable: function(){..},
  zoomTo: function(){..}
}

// could be used somehow like this
var paper = Snap();
paper.zpd.init();
paper.zpd.zoomTo("4x");

In regard to the event system I would still need to look into how Snap.svg handles event internally, but I think it still uses the same system as Raphaeljs http://raphaeljs.com/reference.html#eve. It would probably make sense to harness the functionality already provided.

from snap.svg.zpd.

close as stale discussion, open if there is any plan for the new release / discussion

from snap.svg.zpd.