[Uml-devel] branches/KDE/3.5/kdesdk/umbrello

[Oliver Kellogg]

SVN commit 647909 by okellogg:

mousePressEmpty(): Do not call ToolBarState::mousePressEmpty() on RMB press.

How to use this fix:
1) Make a multiple selection on an existing diagram either by left mouse button
   click on diagram items with the Shift pressed, or by pressing Control-A on
   the keyboard to select all diagram items.
2) Click right mouse button on empty diagram space and at the popup menu select
   "Copy".
3) Go to the target diagram and press the right mouse button in the diagram
   area. At the popup menu, select "Paste".

BUG:143581


 M  +1 -0      ChangeLog  
 M  +6 -1      umbrello/toolbarstatearrow.cpp  


--- branches/KDE/3.5/kdesdk/umbrello/ChangeLog #647908:647909
@@ -31,6 +31,7 @@
   not checked (142093)
 * Missing "with" on Ada code generation for aggregation (142392)
 * Operation Properties "Type" combo box too small (143319)
+* Support duplication of diagrams (143581)
 
 Version 1.5.61
 
--- branches/KDE/3.5/kdesdk/umbrello/umbrello/toolbarstatearrow.cpp #647908:647909
@@ -42,10 +42,15 @@
 }
 
 void ToolBarStateArrow::mousePressEmpty() {
+    if (m_pMouseEvent->button() != Qt::LeftButton) {
+        // Leave widgets selected upon RMB press on empty diagram area.
+        // The popup menu is activated upon RMB release.
+        return;
+    }
     ToolBarState::mousePressEmpty();
 
     // Starts the selection rectangle
-    if (m_pMouseEvent->button() == Qt::LeftButton && m_selectionRect.count() == 0) {
+    if (m_selectionRect.count() == 0) {
         m_startPosition = m_pMouseEvent->pos();
 
         for (int i = 0; i < 4; i++) {