From ba92d4c9227215f5ac033c63bb3ff078dab669eb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thorbj=C3=B8rn=20Lindeijer?= <thorbjorn.lindeijer@nokia.com>
Date: Thu, 17 Mar 2011 15:28:20 +0100
Subject: QmlJSDebugger: Removed the double-click feature from the color box

It was supposed to show a color dialog, but in practice it never worked
due to using show() rather than exec(). Removed it completely since it
was judged confusing and not very useful, since it is already possible
to copy the color by using the context menu.

Task-number: QTCREATORBUG-3878
Reviewed-by: Kai Koehne
---
 src/plugins/qmljsinspector/qmljstoolbarcolorbox.cpp | 7 -------
 src/plugins/qmljsinspector/qmljstoolbarcolorbox.h   | 1 -
 2 files changed, 8 deletions(-)

(limited to 'src')

diff --git a/src/plugins/qmljsinspector/qmljstoolbarcolorbox.cpp b/src/plugins/qmljsinspector/qmljstoolbarcolorbox.cpp
index 6c152185e4..f83aa71f4d 100644
--- a/src/plugins/qmljsinspector/qmljstoolbarcolorbox.cpp
+++ b/src/plugins/qmljsinspector/qmljstoolbarcolorbox.cpp
@@ -124,17 +124,10 @@ void ToolBarColorBox::contextMenuEvent(QContextMenuEvent *ev)
     contextMenu.exec(ev->globalPos());
 }
 
-void ToolBarColorBox::mouseDoubleClickEvent(QMouseEvent *)
-{
-    QColorDialog dialog(m_color);
-    dialog.show();
-}
-
 void ToolBarColorBox::copyColorToClipboard()
 {
     QClipboard *clipboard = QApplication::clipboard();
     clipboard->setText(m_color.name());
 }
 
-
 } // namespace QmlJSInspector
diff --git a/src/plugins/qmljsinspector/qmljstoolbarcolorbox.h b/src/plugins/qmljsinspector/qmljstoolbarcolorbox.h
index 16842fe45a..d53e28b524 100644
--- a/src/plugins/qmljsinspector/qmljstoolbarcolorbox.h
+++ b/src/plugins/qmljsinspector/qmljstoolbarcolorbox.h
@@ -53,7 +53,6 @@ public:
 
 protected:
     void contextMenuEvent(QContextMenuEvent *ev);
-    void mouseDoubleClickEvent(QMouseEvent *);
     void mousePressEvent(QMouseEvent *ev);
     void mouseMoveEvent(QMouseEvent *ev);
 private slots:
-- 
cgit v1.2.1