Code review comment for lp:~bjornt/launchpad/bug-415337

Revision history for this message
Björn Tillenius (bjornt) wrote :

Fix the test_filebug_extra_options windmill test to make it pass again.
The layout of the page had changed a bit, and they way the collapsible
work have changed.

The original test tried to make sure that the extra option fields
weren't shown by checking .clientWidth. This doesn't work anymore, it's
always greater than 0. I changed it to check that the div they are in
is hidden instead, which should be good enough.

--
Björn Tillenius | https://launchpad.net/~bjornt

« Back to merge proposal