Code review comment for lp:~bac/juju-gui/1083933

Revision history for this message
Brad Crittenden (bac) wrote :

Reviewers: mp+143120_code.launchpad.net,

Message:
Please take a look.

Description:
Add tests for new viewport module.

Add tests for the new module. Also removed an unused function. It
looked
useful but I didn't think it should hang around if not called. Ben
might have
had a plan for it.

https://code.launchpad.net/~bac/juju-gui/1083933/+merge/143120

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/7085057/

Affected files:
   A [revision details]
   M app/views/topology/viewport.js
   M test/index.html
   A test/test_viewport_module.js

« Back to merge proposal