yellow team mailing list archive

[Matthew Scott <matthew.scott@xxxxxxxxxxxxx>]

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.