Code review comment for lp://staging/~gary/juju-gui/enableInspectorCancel

Revision history for this message
Benjamin Saller (bcsaller) wrote :

LGTM
QA Good in IE/Chrome

Thanks for this.

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

https://codereview.appspot.com/13677044/diff/1/app/views/inspector.js#newcode837
app/views/inspector.js:837:
this.viewletManager.bindingEngine.resetDOMToModel('config');
This is how I hoped these would read, sorry it was so much work to make
that actually true.

https://codereview.appspot.com/13677044/

« Back to merge proposal