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
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: topology/ viewport. js viewport_ module. js
A [revision details]
M app/views/
M test/index.html
A test/test_