Change mouse cursor to the pointing hand when Ctrl+Shift are pressed

mgrojo · mgrojo · commit f18611ae17b9 · 2022-06-25T01:06:52.000+02:00
This will make the functionality of opening a URL or foreign key more visible. See comment in: #3057 (comment)
diff --git a/src/ExtendedTableWidget.cpp b/src/ExtendedTableWidget.cpp
@@ -962,13 +962,27 @@ void ExtendedTableWidget::keyPressEvent(QKeyEvent* event)
         return;
     }
 
+    // Discoverability of Ctrl+Shift+Click to follow foreign keys or URLs
+    if(event->modifiers().testFlag(Qt::ControlModifier) && event->modifiers().testFlag(Qt::ShiftModifier))
+        QApplication::setOverrideCursor(Qt::PointingHandCursor);
+
     // This prevents the current selection from being changed when pressing tab to move to the next filter. Note that this is in an 'if' condition,
     // not in an 'else if' because this way, when the tab key was pressed and the focus was on the last cell, a new row is inserted and then the tab
     // key press is processed a second time to move the cursor as well
     if((event->key() != Qt::Key_Tab && event->key() != Qt::Key_Backtab) || hasFocus())
         QTableView::keyPressEvent(event);
 }
 
+void ExtendedTableWidget::keyReleaseEvent(QKeyEvent* event)
+{
+
+    // Restore the PointingHandCursor override
+    if(!event->modifiers().testFlag(Qt::ControlModifier) || !event->modifiers().testFlag(Qt::ShiftModifier))
+        QApplication::restoreOverrideCursor();
+
+    QTableView::keyReleaseEvent(event);
+}
+
 void ExtendedTableWidget::updateGeometries()
 {
     // Call the parent implementation first - it does most of the actual logic
diff --git a/src/ExtendedTableWidget.h b/src/ExtendedTableWidget.h
@@ -121,6 +121,7 @@ private slots:
 
 protected:
     void keyPressEvent(QKeyEvent* event) override;
+    void keyReleaseEvent(QKeyEvent* event) override;
     void updateGeometries() override;
     void dragEnterEvent(QDragEnterEvent* event) override;
     void dragMoveEvent(QDragMoveEvent* event) override;
