Add location annotations for service drags.

Add location annotations for service drags.

When a service block is moved, the location annotation is updated when the
drag event ends.

https://code.launchpad.net/~bac/juju-gui/1096273/+merge/142756

(do not edit description out of merge proposal)

[bac, 2013-01-10 18:57:20]

Please take a look.

[gary.poster, 2013-01-10 19:05:09]

Land as is, if Ben confirms that the event registration is OK. :-)

Looks good.  Thank you!

Gary

https://codereview.appspot.com/7070069/diff/1/app/views/topology/service.js
File app/views/topology/service.js (right):

https://codereview.appspot.com/7070069/diff/1/app/views/topology/service.js#n...
app/views/topology/service.js:334: .on('dragend', Y.bind(this._dragend, this));
Please double-check with Ben that this can't be specified as a topology-level
listener, rather than a per-service listener.

[bac, 2013-01-10 20:06:30]

Please take a look.

[bcsaller, 2013-01-10 20:35:52]

LGTM, thanks

https://codereview.appspot.com/7070069/diff/1/app/views/topology/service.js
File app/views/topology/service.js (right):

https://codereview.appspot.com/7070069/diff/1/app/views/topology/service.js#n...
app/views/topology/service.js:267: topo.get('env').update_annotations(d.id, {x:
d.x, y: d.y});
That came out pretty cleanly :)

https://codereview.appspot.com/7070069/diff/1/app/views/topology/service.js#n...
app/views/topology/service.js:334: .on('dragend', Y.bind(this._dragend, this));
On 2013/01/10 19:05:09, gary.poster wrote:
> Please double-check with Ben that this can't be specified as a topology-level
> listener, rather than a per-service listener.

Unfortunately we can't use a top level delegate directly. This approach is fine.
The option we used else where is simply to fire the event on the yui channel and
listen for it there, but in that case you'd still have to resolve which 
service was in play, so again, this is fine.

[bac, 2013-01-10 20:42:05]

*** Submitted:

Add location annotations for service drags.

When a service block is moved, the location annotation is updated when the
drag event ends.

R=gary.poster, bcsaller
CC=
https://codereview.appspot.com/7070069