Hi all,<div><br></div><div>I saw two instances of code duplication in ROShorterDistanceAttachPoint so I refactored the class, changed some method names to make their purpose cleaner and added some comments.</div><div><br></div>

<div>@Alexandre: If you like the changes, could you integrate them into Roassal?</div><div><br></div><div>The filed out class can be found here:</div><div><br></div><div><a href="http://cl.ly/code/392s3y1V2A0N" target="_blank">http://cl.ly/code/392s3y1V2A0N</a></div>

<div><br></div><div>In a next step I added an offset parameter which can be used to offset the attach point positively or negatively on one of the four possible attach &#39;sides&#39;.</div><div>The purpose of the offset can be seen in the following screenshot:</div>
<div><br></div><div><a href="http://cl.ly/image/0d343Z1L0b00">http://cl.ly/image/0d343Z1L0b00</a></div><div><br></div><div>It allows for example to have multiple edges from and to the same element (with different offsets).</div>
<div><br></div><div>I&#39;m note sure whether you would want the offset-possibility in ROShorterDistanceAttachPoint or if I should create a subclass for it. </div><div>If you&#39;d like to integrate it directly, the filed out class with the offset parameter can be found here:<br>
<div><br></div><div><a href="http://cl.ly/code/1T3c310J2r06" target="_blank">http://cl.ly/code/1T3c310J2r06</a></div></div><div><br></div><div>If not I will create a subclass.</div><div><br></div><div>Cheers,</div><div>Dennis</div>