Code review comment for lp:~bcsaller/juju-gui/persistent-layout

Revision history for this message
Francesco Banconi (frankban) wrote :

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