Allow delegate to be instantiated without a root, allow root to be set via prototype method about ftdomdelegate HOT 17 CLOSED

The function to set the dom element of an already instantiated delegate object could be

function watch(root) {
  // TODO: Add the code that is currently in the
  // constructor, then in the constructor call this
  // function if the argument is provided.
}

from ftdomdelegate.

How about attach and detach? The method watch seems to imply that you can keep adding root elements on which to listen for events, but we don't support that.

from ftdomdelegate.

attach also seems to imply that you can keep adding root elements too...

from ftdomdelegate.

You're right. How about pause and play, with play accepting a new root as the argument?

from ftdomdelegate.

If we could use bind and unbind (rebind?) that would be perfect. (jQuery sets up functions with these names - maybe we can too? May get confusing with the native js function bind though)

Pause and play imply you could temporarily stop the delegate (pause();) and then restore it to use its original root element (play();) which I don't think is needed...

(iPhone GitHub app so apologies for no grammar)

from ftdomdelegate.

Pause and play imply you could temporarily stop the delegate (pause();) and then restore it to use its original root element (play();)

I actually like this. bind and unbind are a bit vague - I think even people who use jQuery have to check the documentation every time. In our case it'd be worse because we'd also have on and off.

So we'd have pause and play, which leave the list of registered callbacks unaffected, and destroy which resets everything.

from ftdomdelegate.

Changed my mind. unbind would allow us to discard the reference to the root node, and that's what we want :)

from ftdomdelegate.

We could still implement destroy which would be:-

delegate.prototype.destroy = function () {
  this.unbind();
  this.off();
}

from ftdomdelegate.

I agree with earlier comments. The 'bind'/'unbind' syntax is confusing. If I'm familiar with jQuery I would assume that this was the API (or an alias to the on/off methods) for attaching and removing events. I suggest delegate.root(el); and something like delegate.clearRoot(); or delegate.root('clear');

Feel free to ignore me :)

from ftdomdelegate.

I accept it is a little confusing but I think we can do better than root and clearRoot.

What do we have so far...
watch & unwatch
attach & detach
play & pause
bind & unbind
root & clearRoot

Some more to throw into the mix:-
tie & untie
link & unlink
enslave & liberate / free
connect & disconnect
hold & release
represent & betray
apple & cranberry
marry & divorce (https://twitter.com/TheTeethgrinder/status/293472511788711936)

from ftdomdelegate.

lol! Loving this. Such evocative words for assigning an element to an instance!

What about:

delegate.el(element) // assign
delegate.el() //unassign

from ftdomdelegate.

No hablo español.

setRoot(element) & setRoot() ?

from ftdomdelegate.

Ok. Why not:

delegate.root(element);
delegate.root();

?

from ftdomdelegate.

unroot?

from ftdomdelegate.

root and uproot !!!

from ftdomdelegate.

Too clever! Chill out ;)

I guess unroot works. I'm just alway in favour of keeping the interface as small as possible (if possible). Especially with a lib as small as this.

from ftdomdelegate.

Sold. Let's go with root(element) and root().

from ftdomdelegate.