[umbrello-devel] [umbrello] [Bug 338086] Selection of floating text on diagram is broken

Oliver Kellogg okellogg at users.sourceforge.net
Fri Aug 8 04:12:48 UTC 2014


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

           What    |Removed                     |Added
      Latest Commit|                            |http://commits.kde.org/umbr
                   |                            |ello/69945c9e493eadba4cd8ed
                   |                            |b2c7f15d641cc629d5
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #1 from Oliver Kellogg <okellogg at users.sourceforge.net> ---
Git commit 69945c9e493eadba4cd8edb2c7f15d641cc629d5 by Oliver Kellogg.
Committed on 08/08/2014 at 04:11.
Pushed by okellogg into branch 'master'.

Fix selection of floating text on diagram:

- Move function onWidget() from UMLWidget to WidgetBase.
- If initial dynamic_cast of `this' to UMLWidget* yields NULL then return

- Reimplement WidgetBase function onWidget() in order to check owned
  floating texts.

umbrello/widgets/floatingtextwidget.cpp function onWidget()
- If dynamic_cast of parentItem() to WidgetBase* yields NULL then return
- Admit case wt_Association to setting of isWidgetChild to true.
- If isWidgetChild is false then return WidgetBase::onWidget(p).
- If isWidgetChild is true then finally return NULL if `p' is found to lie
  outside own bounds.

umbrello/umlscene.cpp function widgetAt()
- After iteration over m_WidgetList add iteration over m_AssociationList
  with test of AssociationWidget::onWidget() on each element.

M  +11   -1    umbrello/umlscene.cpp
M  +29   -0    umbrello/widgets/associationwidget.cpp
M  +1    -0    umbrello/widgets/associationwidget.h
M  +23   -20   umbrello/widgets/floatingtextwidget.cpp
M  +0    -28   umbrello/widgets/umlwidget.cpp
M  +0    -2    umbrello/widgets/umlwidget.h
M  +32   -0    umbrello/widgets/widgetbase.cpp
M  +3    -0    umbrello/widgets/widgetbase.h


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

More information about the umbrello-devel mailing list