← Back to team overview

yellow team mailing list archive

Re: Convert charm store to full-side display (issue 6775058)

 

https://codereview.appspot.com/6775058/diff/6001/app/views/charm-search.js
File app/views/charm-search.js (right):

https://codereview.appspot.com/6775058/diff/6001/app/views/charm-search.js#newcode653
app/views/charm-search.js:653: // if (sub) { sub.detach(); }
The only problem I see is that this line is commented out. :)

https://codereview.appspot.com/6775058/

-- 
https://code.launchpad.net/~gary/juju-gui/charmpanel/+merge/131605
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~gary/juju-gui/charmpanel into lp:juju-gui.


References