[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [umbrello] [Bug 128351] Lines connect to bounding box of component interface instead of interface
From:       Ralf Habacker <ralf.habacker () freenet ! de>
Date:       2014-08-14 6:20:28
Message-ID: bug-128351-17878-sGW2YllvOI () http ! bugs ! kde ! org/
[Download RAW message or body]

https://bugs.kde.org/show_bug.cgi?id=128351

--- Comment #24 from Ralf Habacker <ralf.habacker@freenet.de> ---
(In reply to Oliver Kellogg from comment #21)
> Git commit 933143c62f5877bf649ad090d5d34d2e45e2195f by Oliver Kellogg.
> Committed on 10/08/2014 at 22:02.
> Pushed by okellogg into branch 'master'.
> 
> (In reply to Oliver Kellogg from comment #20)
> > Git commit 3360998ea25f5974455939bd100fb4c37bd17d36 by Oliver Kellogg.
> > Committed on 10/08/2014 at 18:28.
> > Pushed by okellogg into branch 'master'.
> > [...]
> > However, apparently Widget_Utils::closestPoints() still has a bug related to
> > anchoring in the NorthWest, West, and SouthWest regions; I'll look into that
> > next.
> 
> Here is the fix for this problem.
> 
With unchecked "angular association line" settings the connected points looks
good (see "testcase for template class association line connections"). You can
see by moving the class "neue_klasse" around the template class, that the shape
is really used.

With above mentioned settings checked the bounding box of the ClassifierWidget
is still used. My guess where to fix is
AssociationWidget::updateRegionLineCount() at the locations where angular != 0. 

{ 
> The association lines now do attach to the template-class widgets.
> However, the attachment points are not always optimal.
> Perhaps the algorithm used by Widget_Utils::closestPoints() can be improved.

With which testcase can I see this ?

-- 
You are receiving this mail because:
You are watching all bug changes.
[prev in list] [next in list] [prev in thread] [next in thread]