[Uml-devel] [umbrello] [Bug 319813] Resizing synchronous message moves its position

[Oliver Kellogg]

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

Oliver Kellogg <okellogg at users.sourceforge.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |okellogg at users.sourceforge.
                   |                            |net

--- Comment #2 from Oliver Kellogg <okellogg at users.sourceforge.net> ---
(In reply to comment #1)
> [...]
> This commit fixes a variant of the related bug when loading message
> widgets from a file. The shadow copy of widgets y position (m_nY)
> has not been set.
> 
> The question is if this shadow copy of widget position is really required
> or if it could be replaced by QGraphicsObject provided widget position.
> 
Many moons ago I had concluded that m_nY is required but I can't remember
exactly why this is so. Furthermore, perhaps the conclusion is outdated.
If you want to try to remove m_nY then please make sure to test the following:
1. Draw creation message
   (This is done by selecting async message and releasing it onto the target
object widget to receive the creation message.)
2. Draw different message types form right to left
  (This is the inverse of the usual left-to-right message flow, and has been
prone to regressions upon code changes in the past)

Thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.