Bug 101541 - More options to select action in sequence diagram
Summary: More options to select action in sequence diagram
Status: RESOLVED FIXED
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Umbrello Development Group
URL:
Keywords:
: 101717 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-03-15 11:52 UTC by Anton G. Alvedro
Modified: 2005-04-16 17:17 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anton G. Alvedro 2005-03-15 11:52:45 UTC
Version:           1.3.92 (using KDE 3.2.3, compiled sources)
Compiler:          gcc version 3.3.3
OS:                Linux (i?86) release 2.2.14-5.0

Although it is straight-forward to change the message interchanged by two classes when a sync/async message is inserted in the sequence diagram, i.e. by right clicking on the arrow and going to properties, it would be handful to be able do do this by double clicking on the arrow as well.

Another possible improvement would be to get the properties popup when drawing a new arrow. You will always need to define what message is interchanged, and by doing it this way, you prevent user to draw inconsistent diagrams (with empty messages). I think this would speed up diagram drawing as well.
Comment 1 Anton G. Alvedro 2005-03-15 14:43:04 UTC
When I am referring to the properties dialog, I mean the Operations dialog.
Comment 2 Oliver Kellogg 2005-04-15 23:57:34 UTC
*** Bug 101717 has been marked as a duplicate of this bug. ***
Comment 3 Oliver Kellogg 2005-04-16 00:08:11 UTC
CVS commit by okellogg: 

mouseRelease(): Showing the operation dialog speeds up message creation.
CCBUG:101541


  M +5 -1      toolbarstatemessages.cpp   1.17


--- kdesdk/umbrello/umbrello/toolbarstatemessages.cpp  #1.16:1.17
@@ -118,8 +118,12 @@ void ToolBarStateMessages::mouseRelease(
                         m_pUMLView->connect(m_pUMLView, SIGNAL(sigColorChanged(Uml::IDType)), message, SLOT(slotColorChanged(Uml::IDType)));
 
-                        message->setActivated();
+                        message->activate();
 
                         m_pSelectedWidget = 0;
                         m_pUMLView->getMessageList().append(message);
+                        FloatingText *ft = message->getFloatingText();
+                        ft->showOpDlg();
+                        message->setTextPosition();
+                        m_pUMLView->getWidgetList().append(ft);
                 }
         }
Comment 4 Oliver Kellogg 2005-04-16 17:17:06 UTC
CVS commit by okellogg: 

BUG:101541 - onWidget():
Fix to {top,bottom}ArrowY brings on the properties dialog upon double clicking.


  M +4 -1      ChangeLog   1.75
  M +3 -3      umbrello/messagewidget.cpp   1.71


--- kdesdk/umbrello/ChangeLog  #1.74:1.75
@@ -2,8 +2,10 @@
 
 * Externalization of folders (i.e. submodel files)
-  http://bugs.kde.org/87252
 
+* Bugs fixed / wishes implemented (see http://bugs.kde.org)
+87252 
 
 Version 1.4.1 (maintenance release)
+
 Bugs fixed:
 * Crash on deleting attributes / enum literals
@@ -11,4 +13,5 @@
 * Bugs from http://bugs.kde.org :
 53376 57667 57875 95353 100290 100307 101148 103123 103133 103728
+101541
 
 Version 1.4

--- kdesdk/umbrello/umbrello/messagewidget.cpp  #1.70:1.71
@@ -266,8 +266,8 @@ bool MessageWidget::onWidget(const QPoin
         if (p.x() < getX() || p.x() > getX() + getWidth())
                 return false;
-        const int tolerance = 4;  // pixels
+        const int tolerance = 5;  // pixels
         const int pY = p.y();
-        const int topArrowY = getY();
-        const int bottomArrowY = topArrowY + getHeight();
+        const int topArrowY = getY() + 3;
+        const int bottomArrowY = getY() + getHeight() - 3;
         if (pY < topArrowY - tolerance || pY > bottomArrowY + tolerance)
                 return false;