Completes the scale up UX

Completes the scale up UX

This branch adds the 'edit' step of the scale up
story. The constraint fields on the overview page
are databound so any changes to them will update 
behind the scenes.

https://code.launchpad.net/~hatch/juju-gui/scale-up-pt2/+merge/181620

(do not edit description out of merge proposal)

[jeff.pihach, 2013-08-22 16:54:44]

Please take a look.

[rharding, 2013-08-22 17:01:57]

LGTM with note below.

https://codereview.appspot.com/12808048/diff/1/app/views/inspector.js
File app/views/inspector.js (right):

https://codereview.appspot.com/12808048/diff/1/app/views/inspector.js#newcode155
app/views/inspector.js:155: this.overviewConstraintsEdit = false;
should this be set to some default on the object instantiation?

[jeff.pihach, 2013-08-22 17:10:02]

Thanks for the review!

https://codereview.appspot.com/12808048/diff/1/app/views/inspector.js
File app/views/inspector.js (right):

https://codereview.appspot.com/12808048/diff/1/app/views/inspector.js#newcode155
app/views/inspector.js:155: this.overviewConstraintsEdit = false;
I was thinking that, but because this is a 'mixin' it would be another case of a
property being set in some other class then so this at least keeps it contained
to the same file.

[matthew.scott, 2013-08-22 17:24:25]

LGTM - IE QA was okay, once caches were cleared.  Several times.  *fistshake*

https://codereview.appspot.com/12808048/diff/1/test/test_inspector_overview.js
File test/test_inspector_overview.js (right):

https://codereview.appspot.com/12808048/diff/1/test/test_inspector_overview.j...
test/test_inspector_overview.js:24: 
Trivial: extra newline?

[jeff.pihach, 2013-08-22 17:32:51]

Thanks for the reviews!

[jeff.pihach, 2013-08-22 17:39:39]

*** Submitted:

Completes the scale up UX

This branch adds the 'edit' step of the scale up
story. The constraint fields on the overview page
are databound so any changes to them will update 
behind the scenes.

R=rharding, matthew.scott
CC=
https://codereview.appspot.com/12808048