Looks good Matt.  Thanks.

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

https://codereview.appspot.com/6856067/diff/1/app/views/environment.js#newcode323
app/views/environment.js:323: })
maybe this would be more readable:

min(1, d.unit_count)

https://codereview.appspot.com/6856067/

-- 
https://code.launchpad.net/~makyo/juju-gui/adding-subordinates/+merge/135022
Your team Juju GUI Hackers is requested to review the proposed merge of 
lp:~makyo/juju-gui/adding-subordinates into lp:juju-gui.

-- 
Mailing list: https://launchpad.net/~yellow
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~yellow
More help   : https://help.launchpad.net/ListHelp

Reply via email to