Thanks, Brad.
https://codereview.appspot.com/6846060/diff/1/app/views/environment.js File app/views/environment.js (right): https://codereview.appspot.com/6846060/diff/1/app/views/environment.js#newcode456 app/views/environment.js:456: .removeClass('yui5-button'); On 2012/11/15 20:27:29, bac wrote: > You discovered this typo when we chatted. Done. https://codereview.appspot.com/6846060/diff/1/test/test_environment_view.js File test/test_environment_view.js (right): https://codereview.appspot.com/6846060/diff/1/test/test_environment_view.js#newcode410 test/test_environment_view.js:410: panel.one('button').simulate('click'); On 2012/11/15 20:27:29, bac wrote: > How do we know this happened? Ah, good catch, assertion added. https://codereview.appspot.com/6846060/ -- https://code.launchpad.net/~makyo/juju-gui/no-remove-sub-rels/+merge/134495 Your team Juju GUI Hackers is requested to review the proposed merge of lp:~makyo/juju-gui/no-remove-sub-rels 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
