From c37ce664f5d59d25503a810f0d3212fdb5cb19b7 Mon Sep 17 00:00:00 2001
From: Tyler Goodlet <jgbt@protonmail.com>
Date: Fri, 13 Aug 2021 13:49:07 -0400
Subject: [PATCH] Uggh, don't override click handler on view; pg has it's own
 mouse events..

---
 piker/ui/_interaction.py | 32 ++++++++++++++++++++------------
 1 file changed, 20 insertions(+), 12 deletions(-)

diff --git a/piker/ui/_interaction.py b/piker/ui/_interaction.py
index 88c08359..a01c03ee 100644
--- a/piker/ui/_interaction.py
+++ b/piker/ui/_interaction.py
@@ -23,6 +23,7 @@ import time
 from typing import Optional, Callable
 
 import pyqtgraph as pg
+from pyqtgraph.GraphicsScene import mouseEvents
 from PyQt5.QtWidgets import QGraphicsSceneMouseEvent as gs_mouse
 from PyQt5.QtCore import Qt, QEvent
 from pyqtgraph import ViewBox, Point, QtCore
@@ -250,17 +251,24 @@ async def handle_viewmode_mouse(
 
     async for msg in recv_chan:
         button = msg.button
-        # pos = ev.pos()
 
-        if button == QtCore.Qt.RightButton and view.menuEnabled():
-            # ev.accept()
-            view.raiseContextMenu(msg.event)
+        # XXX: ugggh ``pyqtgraph`` has its own mouse events..
+        # so we can't overried this easily.
+        # it's going to take probably some decent
+        # reworking of the mouseClickEvent() handler.
 
-        elif button == QtCore.Qt.LeftButton:
+        # if button == QtCore.Qt.RightButton and view.menuEnabled():
+        #     event = mouseEvents.MouseClickEvent(msg.event)
+        #     # event.accept()
+        #     view.raiseContextMenu(event)
+
+        if (
+            view.order_mode and
+            button == QtCore.Qt.LeftButton
+        ):
             # when in order mode, submit execution
-            if view.order_mode:
-                # ev.accept()
-                view.mode.submit_order()
+            # msg.event.accept()
+            view.mode.submit_order()
 
 
 class ChartView(ViewBox):
@@ -515,10 +523,10 @@ class ChartView(ViewBox):
             self.scaleBy(x=x, y=y, center=center)
             self.sigRangeChangedManually.emit(self.state['mouseEnabled'])
 
-    def mouseClickEvent(self, event: QtCore.QEvent) -> None:
-        '''This routine is rerouted to an async handler.
-        '''
-        pass
+    # def mouseClickEvent(self, event: QtCore.QEvent) -> None:
+    #      '''This routine is rerouted to an async handler.
+    #      '''
+    #     pass
 
     def keyReleaseEvent(self, event: QtCore.QEvent) -> None:
         '''This routine is rerouted to an async handler.