Hi Ben, thanks for this branch. I have not so much to add to Gary's review. Please see just the one comment below.
https://codereview.appspot.com/7132061/diff/6001/app/views/topology/service.js File app/views/topology/service.js (right):
https://codereview.appspot.com/7132061/diff/6001/app/views/topology/service.js#newcode514 app/views/topology/service.js:514: topo.fire('serviceMoved', {service: d}); Isn't serviceMoved already fired by self.drag?
https://codereview.appspot.com/7132061/
« Back to merge proposal
Hi Ben, thanks for this branch.
I have not so much to add to Gary's review. Please see just the one
comment below.
https:/ /codereview. appspot. com/7132061/ diff/6001/ app/views/ topology/ service. js topology/ service. js (right):
File app/views/
https:/ /codereview. appspot. com/7132061/ diff/6001/ app/views/ topology/ service. js#newcode514 topology/ service. js:514: topo.fire( 'serviceMoved' , {service:
app/views/
d});
Isn't serviceMoved already fired by self.drag?
https:/ /codereview. appspot. com/7132061/