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

List:       kde-commits
Subject:    [umbrello/KDE/4.11] umbrello: Fix of 'Diagram popup menu not displayed at cursor position'.
From:       Ralf Habacker <ralf.habacker () freenet ! de>
Date:       2013-11-30 21:29:28
Message-ID: E1Vms6S-0007n7-M0 () scm ! kde ! org
[Download RAW message or body]

Git commit 51b3164ec8d61d6d9eb796157f4d5a3075e073a3 by Ralf Habacker.
Committed on 30/11/2013 at 18:20.
Pushed by habacker into branch 'KDE/4.11'.

Fix of 'Diagram popup menu not displayed at cursor position'.

The problem has been caused by handling the diagram context menu on
two source code locations, which results into a race condition.

BUG:328257
FIXED-IN:4.11.5
(cherry picked from commit 362c2a36fcdd4b2649b3905081552352d7ee1a4c)

M  +0    -2    umbrello/toolbarstatearrow.cpp

http://commits.kde.org/umbrello/51b3164ec8d61d6d9eb796157f4d5a3075e073a3

diff --git a/umbrello/toolbarstatearrow.cpp b/umbrello/toolbarstatearrow.cpp
index 7c16f60..aaea28b 100644
--- a/umbrello/toolbarstatearrow.cpp
+++ b/umbrello/toolbarstatearrow.cpp
@@ -125,8 +125,6 @@ void ToolBarStateArrow::mouseReleaseEmpty()
         while (!m_selectionRect.isEmpty())
             delete m_selectionRect.takeFirst();
         m_selectionRect.clear();
-    } else if (m_pMouseEvent->button() == Qt::RightButton) {
-        m_pUMLScene->setMenu(m_pMouseEvent->scenePos().toPoint());
     }
 }
 
[prev in list] [next in list] [prev in thread] [next in thread]