Code review comment for lp:~jcsackett/juju-gui/search-fixes

Revision history for this message
Jeff Pihach (hatch) wrote :

LGTM thanks, with trivial

https://codereview.appspot.com/8940043/diff/3001/app/subapps/browser/browser.js
File app/subapps/browser/browser.js (right):

https://codereview.appspot.com/8940043/diff/3001/app/subapps/browser/browser.js#newcode24
app/subapps/browser/browser.js:24: * Show or hide the details panel.
We decided against the *'s in a Friday call.

https://codereview.appspot.com/8940043/diff/3001/test/test_browser_app.js
File test/test_browser_app.js (right):

https://codereview.appspot.com/8940043/diff/3001/test/test_browser_app.js#newcode514
test/test_browser_app.js:514: it('no querystring still searched',
function() {
non descriptive test name.

https://codereview.appspot.com/8940043/

« Back to merge proposal