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

Revision history for this message
Gary Poster (gary) wrote :

On 2013/09/13 20:37:20, benjamin.saller 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.

Thank you, Ben!

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

« Back to merge proposal