juju-gui

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

  1. bug1204331
  2. Merge into trunk
Revision history for this message
Brad Crittenden (bac) wrote :

LGTM for code. QA shows the dismiss [X] on the inspector has dropped
50px or so out of place. The issues you meant to solve are fixed.

https://codereview.appspot.com/11804043/diff/1/app/index.html
File app/index.html (right):

https://codereview.appspot.com/11804043/diff/1/app/index.html#newcode70
app/index.html:70: <body>
I'm glad you resolved this issue.

https://codereview.appspot.com/11804043/diff/1/app/models/charm.js
File app/models/charm.js (right):

https://codereview.appspot.com/11804043/diff/1/app/models/charm.js#newcode178
app/models/charm.js:178: // fakebackend and/or sandbox to send the
expected thing there.
Trivial but this should be an XXX with your name on it.

https://codereview.appspot.com/11804043/

« Back to merge proposal