Mouse interaction for completion popup

src/calibre/gui2/tweak_book/completion/popup.py

@@ -29,6 +29,8 @@ def __init__(self, parent, max_height=1000):
         self.setFocusPolicy(Qt.NoFocus)
         self.setFocusProxy(parent)
         self.setVisible(False)
+        self.setMouseTracking(True)
+        self.setCursor(Qt.PointingHandCursor)
 
         self.matcher = None
         self.current_results = self.current_size_hint = None
@@ -98,6 +100,11 @@ def iter_visible_items(self):
             yield i + self.current_top_index, st, y, height
             y += height
 
+    def index_for_y(self, y):
+        for idx, st, top, height in self.iter_visible_items():
+            if top <= y < top + height:
+                return idx
+
     def paintEvent(self, ev):
         painter = QPainter(self)
         painter.setClipRect(ev.rect())
@@ -214,6 +221,9 @@ def handle_keypress(self, ev):
         if key in (Qt.Key_Up, Qt.Key_Down):
             self.choose_next_result(previous=key == Qt.Key_Up)
             return True
+        if key in (Qt.Key_Enter, Qt.Key_Return) and self.current_index > -1:
+            self.index_activated(self.current_index)
+            return True
         return False
 
     def eventFilter(self, obj, ev):
@@ -228,6 +238,25 @@ def eventFilter(self, obj, ev):
                 self.relayout_timer.start()
         return False
 
+    def mouseMoveEvent(self, ev):
+        y = ev.pos().y()
+        idx = self.index_for_y(y)
+        if idx is not None and idx != self.current_index:
+            self.current_index = idx
+            self.update()
+            ev.accept()
+
+    def mouseReleaseEvent(self, ev):
+        y = ev.pos().y()
+        idx = self.index_for_y(y)
+        if idx is not None:
+            self.index_activated(idx)
+        ev.accept()
+
+    def index_activated(self, idx):
+        print ('index activated', idx)
+        self.hide()
+
 if __name__ == '__main__':
     from calibre.utils.matcher import Matcher
     def test(editor):

src/calibre/gui2/tweak_book/editor/text.py

@@ -662,6 +662,10 @@ def link_for_position(self, pos):
             return self.text_for_range(c.block(), r)
 
     def mousePressEvent(self, ev):
+        if self.completion_popup.isVisible() and not self.completion_popup.rect().contains(ev.pos()):
+            # For some reason using eventFilter for this does not work, so we
+            # implement it here
+            self.completion_popup.hide()
         if ev.modifiers() & Qt.CTRL:
             url = self.link_for_position(ev.pos())
             if url is not None: