Index: lams_central/web/includes/javascript/authoring/authoringActivity.js =================================================================== diff -u -rc6dfdf0a0370d091b4fb697afa2f151ba9fa56fa -r070fb3365c9a8186b44a1b442da4866577954cb1 --- lams_central/web/includes/javascript/authoring/authoringActivity.js (.../authoringActivity.js) (revision c6dfdf0a0370d091b4fb697afa2f151ba9fa56fa) +++ lams_central/web/includes/javascript/authoring/authoringActivity.js (.../authoringActivity.js) (revision 070fb3365c9a8186b44a1b442da4866577954cb1) @@ -1350,10 +1350,11 @@ findTransitionPoints : function(fromActivity, toActivity) { var fromActivityBox = fromActivity.items.getBBox(), toActivityBox = toActivity.items.getBBox(), - horizontalDelta = Math.abs(fromActivityBox.cx - toActivityBox.cx), - verticalDelta = Math.abs(fromActivityBox.cy - toActivityBox.cy), - // if the box is more up/down then left/right, then arrow direction is vertical - direction = horizontalDelta > verticalDelta ? 'horizontal' : 'vertical', + // vertical direction takes priority + // horizontal is used only if activities are in the same line + direction = (fromActivityBox.y > toActivityBox.y && fromActivityBox.y < toActivityBox.y2) + || (fromActivityBox.y2 > toActivityBox.y && fromActivityBox.y2 < toActivityBox.y2) + ? 'horizontal' : 'vertical', points = null; if (direction === 'vertical') {