Fix constraints save and add save confirmation

THIS PROTOTYPE BRANCH IS CLOSED; WORK IS BEING LANDED GRADUALLY, ELSEWHERE.


Fix constraints save and add save confirmation

To QA, deploy a service with the inspector, and then reopen deployed inspector.  In configuration and constraints, make changes and click save.  The save button should now not change text when you click on it in the constraints tab. The save (and cancel) buttons only appear when there are changes to be saved.  The fields correctly recognize when changes are reverted.  When changes are saved, the field flashes green to show success.

checkboxes do not work properly in this branch: Rick is working on that.  The cancel button does not work: that will be a separate branch.

For comment and discussion, code and UX.  The green flash for fields on save is not as described by UX, but the desired display will be a lot more time consuming--and fragile, I am afraid--than what I have here.  I am hoping this is a reasonable compromise.

I hope discussion will answer UX issues, as well as resolve what is appropriate for testing.  I am a bit too focused on getting this out the door, so need some outside perspective.

https://code.launchpad.net/~gary/juju-gui/fixsavetext/+merge/183996

(do not edit description out of merge proposal)

[gary.poster, 2013-09-05 00:45:25]

Please take a look.

[jeff.pihach, 2013-09-05 14:12:50]

Looking good but there is some duplication of work, plz see below.

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

https://codereview.appspot.com/13505044/diff/1/app/views/inspector.js#newcode...
app/views/inspector.js:1047: if (!this.modifiedKeys) {
When a user changes the value of an input the databinding engine keeps track of
these changes via it's _storeChanged method on line 609 of databinding.js. This
method keeps track of these changes in each viewlets _changedvalues method. It
would be nice if you could use this data set instead of trying to keep track of
it yourself. If you need reference to these values in this changed method you
may need to pass it into the method call on line 622.