yellow team mailing list archive

[Matthew Scott <matthew.scott@xxxxxxxxxxxxx>]

The proposal to merge lp:~makyo/juju-gui/bad-drag into lp:juju-gui has been updated.

Description changed to:

Fix for rounding error on drag-relation.

New service blocks have integer x/y coordinates, leading to mouseenter failing at the d.containsPoint() test when one of the mouse coordinates matched exactly with one of the edges of the service block.  To reproduce, create two new services (say varnish and node-app) and, without navigating away from the environment view, drag them to be diagonal relative to each other, then try to drag a relation from one to the other through the corner of the service node.  Correct behavior (this branch) should be for the drag line to snap to the two nearest connectors (as opposed to passing through the service without recognizing that it had entered).

https://codereview.appspot.com/6741047/


For more details, see:
https://code.launchpad.net/~makyo/juju-gui/bad-drag/+merge/130424
-- 
https://code.launchpad.net/~makyo/juju-gui/bad-drag/+merge/130424
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~makyo/juju-gui/bad-drag into lp:juju-gui.